Comment 11 for bug 911146
Revision history for this message
| John Paul Adrian Glaubitz (glaubitz) wrote | #11 |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
bbfa235
demo site
(Get the code!)
> AFAICT, 0.48.4 and trunk are no longer affected by bug #695120 and the regression (bug #1022719) is fixed.
> But the patch looks more elegant than the one we applied, and I would be interesting to test it.
The patch by Michael Karcher actually fixes both bug #695120 and avoids the regression bug #1022719 introduced by the fix for bug #695120. Both patches in bug #695120 and bug #1022719 can therefore be replaced with this single patch.
Michael's patch relies on GLib for testing for relative pathnames and building an absolute one if necessary. Thus, there is no need to worry about (invalid) pathnames and URLs since everything is handled by GLib. The patches that have been applied so far duplicate the code in GLib.
Cheers,
Adrian