[wpkg-users] [Bug 146] "remove" action should set environment from profiles/hosts

[bugzilla-daemon at bugzilla.wpkg.org]

http://bugzilla.wpkg.org/show_bug.cgi?id=146


Rainer Meier <r.meier at wpkg.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED




--- Comment #3 from Rainer Meier <r.meier at wpkg.org>  2009-04-10 21:11:42 ---
Hi Jason (and others),

I had a look at the patch and with some modification I've applied it.
Especially I've modified the scope of the try-catch block slightly to prevent
catching exceptions from dependency removals. In addition I've fixed the output
which was using the "installType" variable which is unavailable for the
removePackage() function.

Moreover I did a small change to the exception handler to prevent reloading the
previous environment twice (well, this also happened on package installation
where I fixed it too).

Please find version 1.1.0-M17 checked in:

<http://wpkg.svn.sourceforge.net/viewvc/wpkg/wpkg/current-development/wpkg.js?view=log>

It's not yet very well tested, probably you could do a quick verification too.


FIX: Fixed loading of package-specific environment for remove commands.
     Previously environment variables set for a package have been unavailable
     to remove commands.
     Fixes Bug 146. Thanks to Jason Oster.


br,
Rainer


-- 
Configure bugmail: http://bugzilla.wpkg.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.