Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=547655 --- Comment #3 from Xavier Bachelot <xavier@xxxxxxxxxxxx> 2009-12-16 03:51:34 EDT --- > > * Can you explain your patch? > > The original code first searches @INC, then searches the other (IMO > > questionable) places. As this module's purpose is to first check where RT is > > installed on a system, your patch IMO is wrong. > > > The problem is it tries to use LocalPath from the @INC RT.pm, which is set to > /usr/local/lib/rt3, thus installing the extension to the wrong place. So I > create another RT.pm for the installation purpose and force the use of it thru > the RTHOME envvar. However, the RT.pm from @INC is preferred over the one from > RTHOME, which is a dubious in my opinion. If one is forcing the use of a > RTHOME, one would assume this will be the first tried location, hence the > patch. > Oops, sorry, this is indeed not understandable as is. I'll blame a missing morning coffee. The RTHOME usage is actually referring to the perl-RTx-Calendar rpackage (https://bugzilla.redhat.com/show_bug.cgi?id=493335), which is making use of Module::Install::RTx. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review