[virt-tools-list] virtinst-0.500.6 question, clean build environment issues

[Cole Robinson]

On 07/17/2011 08:27 PM, Jason Helfman wrote:
> Hi Cole,
> 
> I've applied this patch, and ran it again, and the same thing happens:
> 
> I applied the patch as it was raised by someone at FreeBSD.
> 
> Here is the end of the log:
> 

Okay. That patch disabled all translation file handling in the setup.py build
file, so I assume the issue is on the freebsd side.

- Cole

> ===> Building package for py27-virtinst-0.500.6_1
> Deleting py27-virtinst-0.500.6_1
> === Checking filesystem state
> Deleting py27-urlgrabber-3.9.1
> Deleting libvirt-0.9.3
> Deleting gnutls-2.12.7
> Deleting libxml2-2.7.8_1
> Deleting py27-curl-7.19.0_1
> Deleting python27-2.7.1_1
> Deleting pkg-config-0.25_1
> Deleting libgcrypt-1.4.6
> Deleting curl-7.21.3_2
> Deleting ca_root_nss-3.12.9
> Deleting libgpg-error-1.10
> Deleting gettext-0.18.1.1
> Deleting libiconv-1.13.1_1
> === Checking filesystem state after all packages deleted
> ================================================================
> list of files present on clean system but missing after everything was
> deinstalled)
> ./usr/local/share/locale/he missing
> ./usr/local/share/locale/he/LC_MESSAGES missing
> ./usr/local/share/locale/hr missing
> ./usr/local/share/locale/hr/LC_MESSAGES missing
> ./usr/local/share/locale/lt missing
> ./usr/local/share/locale/lt/LC_MESSAGES missing
> ================================================================
> build of /usr/ports/net-mgmt/virtinst ended at Mon Jul 18 00:19:30 UTC 2011
> 
> Thanks,
> Jason
> 
> On Tue, Jul 12, 2011 at 10:23:30AM -0400, Cole Robinson thus spake:
>> On 07/08/2011 05:26 PM, Richard W.M. Jones wrote:
>>> On Fri, Jul 08, 2011 at 02:17:08PM -0700, Jason Helfman wrote:
>>>> Right. Here is a link to the current bsd.port.mk file.
>>>> http://www.freebsd.org/cgi/cvsweb.cgi/ports/Mk/bsd.port.mk?rev=1.687;content-type=text%2Fx-cvsweb-markup
>>>>
>>>
>>> It appears to run:
>>>
>>>   pkg_delete -f $name
>>>
>>> and pkg_delete (according to [1]) looks very much like RPM and other
>>> packaging tools, ie. sensible.
>>>
>>> However I'm not any closer to understanding why those locale files
>>> would be deleted after the deinstall.  It appears to be some
>>> interaction between the pkg_* tools and files that virt-install
>>> modifies, but in what way I don't know.
>>>
>>> Rich.
>>>
>>> [1] http://www.freebsd.org/cgi/man.cgi?query=pkg_delete&sektion=1
>>>
>>
>> Thanks for looking into it Rich, doesn't make much sense to me either,
>> nothing in the virtinst build process seems to be touching those files.
>>
>> Jason, does applying the following patch to virtinst make any difference
>> (it just disables all .po building and installing). If not, something
>> must be funky on the bsd side, otherwise maybe some side effect of the
>> po commands we run is causing the bsd package tool to prune those files
>> on uninstall?
>>
>> Thanks,
>> Cole
>>
>> diff --git a/setup.py b/setup.py
>> index 659ddd4..c708bdc 100755
>> --- a/setup.py
>> +++ b/setup.py
>> @@ -277,6 +277,7 @@ class build(_build):
>>         if not os.path.exists("build/po"):
>>             os.makedirs("build/po")
>>
>> +        """
>>         for filename in glob(pjoin(os.getcwd(), 'po', '*.po')):
>>             filename = os.path.basename(filename)
>>             lang = os.path.basename(filename)[0:len(filename) - 3]
>> @@ -286,6 +287,7 @@ class build(_build):
>>
>>             print "Building %s from %s" % (newname, filename)
>>             os.system("msgfmt po/%s -o %s" % (filename, newname))
>> +        """
>>
>>         _build.run(self)
>>         builddir = self.build_lib
>> @@ -317,6 +319,7 @@ class install_data(_install_data):
>>     """ custom install_data command to prepare i18n files for install"""
>>
>>     def run(self):
>> +        """
>>         dirlist = os.listdir("build/po")
>>         for lang in dirlist:
>>             if lang != "." and lang != "..":
>> @@ -329,6 +332,7 @@ class install_data(_install_data):
>>
>>                 # Add these to the datafiles list
>>                 datafiles.append(toadd)
>> +        """
>>         _install_data.run(self)
>>
>> setup(name='virtinst',
>>
>