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

[Jason Helfman]

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',
>

I didn't apply this patch, because it is thought of at this point (and
proven) that something odd is happening inside of the jail on the build
cluster. If this gets fixes, I will let you know.

Thanks,
Jason

-- 
Jason Helfman
System Administrator
experts-exchange.com
http://www.experts-exchange.com/M_4830110.html
E4AD 7CF1 1396 27F6 79DD  4342 5E92 AD66 8C8C FBA5