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


Rainer Meier <r.me...@wpkg.org> changed:

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




--- Comment #3 from Rainer Meier <r.me...@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.
-------------------------------------------------------------------------
wpkg-users mailing list archives >> http://lists.wpkg.org/pipermail/wpkg-users/
_______________________________________________
wpkg-users mailing list
wpkg-users@lists.wpkg.org
http://lists.wpkg.org/mailman/listinfo/wpkg-users

Reply via email to