Bug 4401 - @INC no longer includes Plugins Dir.
: @INC no longer includes Plugins Dir.
Status: RESOLVED DUPLICATE of bug 4112
Product: Logitech Media Server
Classification: Unclassified
Component: Plugins
: 6.5.1
: PC Windows (legacy)
: P2 normal (vote)
: ---
Assigned To: Dan Sully
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2006-10-20 09:44 UTC by KDF
Modified: 2006-10-30 20:48 UTC (History)
1 user (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description KDF 2006-10-20 09:44:44 UTC
I think change 7725 caused it.  dirsFor would correctly return the Plugins directory, but it's stripped off.  I don't understand the comment well enough to know why the change was made or how hard it would be to get this function back with the new plugin api coming up
Comment 1 Dan Sully 2006-10-30 11:07:12 UTC
Why would you want Plugins in the path? If the namespace is: Plugins::Foo, you want to include the directory one above Plugins/, and not Plugins/ itself.

What's broken here that you're trying to fix?
Comment 2 KDF 2006-10-30 11:20:43 UTC
having sat with this a while, I can recognise that this is the desirable behaviour.  It is, however, an apparent change.  It came to my attention when users started complaining about the email plugin, which is packaged with Net::POP3 in the plugins tree.  it's stupid, I know, but so is the plugin so I do as little as possible with it until I can come up with a reasonable excuse to just make it vanish.

However, useful to note that there are warnings coming from the tendancy of now several plugins to include CPAN or other share modules in their packages with similarly unkempt namespaces.  For example, Trackstat 'use's Plugins::TrackStat::Time::Stopwatch (copied from Scrobbler, no doubt), but the Stopwatch module uses the namespace Time::Stopwatch.

I know this report is basically invalid, but I left it here as a reminder to bring this problem up as part of the plugin api changes.  How to solve this issue should be in there.
Comment 3 KDF 2006-10-30 20:48:39 UTC

*** This bug has been marked as a duplicate of 4112 ***