Arch Linux

Please read this before reporting a bug:
https://wiki.archlinux.org/title/Bug_reporting_guidelines

Do NOT report bugs when a package is just outdated, or it is in the AUR. Use the 'flag out of date' link on the package page, or the Mailing List.

REPEAT: Do NOT report bugs for outdated packages!
Tasklist

FS#9491 - mplayer-plugin $MOZ_PLUGIN_PATH

Attached to Project: Arch Linux
Opened by Ray Rashif (schivmeister) - Thursday, 07 February 2008, 23:24 GMT
Last edited by Alexander Baldeck (kth5) - Wednesday, 13 February 2008, 12:00 GMT
Task Type Bug Report
Category Packages: Extra
Status Closed
Assigned To Alexander Baldeck (kth5)
Architecture All
Severity Low
Priority Normal
Reported Version 2007.08-2
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 1
Private No

Details

Package:
mplayer-plugin 3.50-1

Issue:
The variable in question (as per summary), used in the packaging process, has a certain flaw where if the user's rendition of the path includes more than one directory, any separation character (in this case ":") will be captured. This leads to the creation of a redundant file/directory with the character being part of the name, among other potential problems.

Case:
>>$ echo $MOZ_PLUGIN_PATH
>>$ /opt/mozilla/lib/plugins:/usr/lib/mozilla/plugins

So from line 25 of the PKGBUILD,

mkdir -p $startdir/pkg/${MOZ_PLUGIN_PATH}

the user will have an "/opt/mozilla/lib/plugins:" directory. This produces a further redundancy as currently the packager only installs essential files to a specific directory. From PKGBUILD line 31,

install -m755 mplayerplug-in*so ${startdir}/pkg/usr/lib/mozilla/plugins

Thus, "/opt/mozilla/lib/plugins:" and any other paths involved in $MOZ_PLUGIN_PATH will be empty.
This task depends upon

Closed by  Alexander Baldeck (kth5)
Wednesday, 13 February 2008, 12:00 GMT
Reason for closing:  Fixed
Comment by Travis Willard (Cerebral) - Friday, 08 February 2008, 05:47 GMT
  • Assignment removed
Confirmed, but I can't seem to assign it to kth5. :/
Comment by Travis Willard (Cerebral) - Friday, 08 February 2008, 06:32 GMT
aha! Assigned
Comment by Vinay S Shastry (shastry) - Tuesday, 12 February 2008, 17:57 GMT
  • Field changed: Percent Complete (100% → 0%)
The latest package is further fubarred.

Please don't use MOZ_PLUGIN_PATH at all in the PKGBUILD!
Comment by Ray Rashif (schivmeister) - Tuesday, 12 February 2008, 18:07 GMT
Partly my fault for not making it clear. I feel the need to correct myself.

It's just that the value of $MOZ_PLUGIN_PATH will be taken as one path. So you should probably just install directly to /usr (better, since we're on transition) or /opt. At least browsers like Firefox, Flock, and Opera can automatically detect those, but not this faulty one. Even if they do, there may be other issues. Well it's just something minor but can't be ignored either :)
Comment by crc32 (crc64) - Wednesday, 13 February 2008, 06:53 GMT
> Comment by Vinay S Shastry (shastry) - Tuesday, 12 February 2008, 17:57 GMT
> The latest package is further fubarred.

Same here. mplayer-plugin 3.50-2 is worse than 3.50-1 :(

[crc64@Arch pkg]$ tar -tf mplayer-plugin-3.50-2-i686.pkg.tar.gz
.PKGINFO
.INSTALL
etc/
etc/mplayerplug-in.conf
opt/
opt/mozilla/
opt/mozilla/lib/
opt/mozilla/lib/plugins:/
opt/mozilla/lib/plugins:/usr/
opt/mozilla/lib/plugins:/usr/lib/
opt/mozilla/lib/plugins:/usr/lib/mozilla/
opt/mozilla/lib/plugins:/usr/lib/mozilla/plugins/
opt/mozilla/lib/plugins:/usr/lib/mozilla/plugins/mplayerplug-in-wmp.xpt
opt/mozilla/lib/plugins:/usr/lib/mozilla/plugins/mplayerplug-in-qt.xpt
opt/mozilla/lib/plugins:/usr/lib/mozilla/plugins/mplayerplug-in-gmp.so
opt/mozilla/lib/plugins:/usr/lib/mozilla/plugins/mplayerplug-in-rm.so
opt/mozilla/lib/plugins:/usr/lib/mozilla/plugins/mplayerplug-in-gmp.xpt
opt/mozilla/lib/plugins:/usr/lib/mozilla/plugins/mplayerplug-in.xpt
opt/mozilla/lib/plugins:/usr/lib/mozilla/plugins/mplayerplug-in-wmp.so
opt/mozilla/lib/plugins:/usr/lib/mozilla/plugins/mplayerplug-in-dvx.so
opt/mozilla/lib/plugins:/usr/lib/mozilla/plugins/mplayerplug-in-qt.so
opt/mozilla/lib/plugins:/usr/lib/mozilla/plugins/mplayerplug-in.so
opt/mozilla/lib/plugins:/usr/lib/mozilla/plugins/mplayerplug-in-rm.xpt
opt/mozilla/lib/plugins:/usr/lib/mozilla/plugins/mplayerplug-in-dvx.xpt
usr/
usr/lib/
usr/lib/mozilla/
usr/lib/mozilla/plugins/


Comment by Simo Leone (neotuli) - Wednesday, 13 February 2008, 09:43 GMT
perhaps we should just hardcode /usr/lib/mozilla/plugins, since all the mozilla stuff is supposed to move out of /opt eventually anyway.

If you don't get around to it by like tomorrow, I can take care of it.
Comment by Alexander Baldeck (kth5) - Wednesday, 13 February 2008, 11:59 GMT
Done already. So yeah, fixed.

Loading...