Bug 151270 - [PATCH] multimedia/xfce4-parole: explicitly disable browser plugin
Summary: [PATCH] multimedia/xfce4-parole: explicitly disable browser plugin
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Martin Wilke
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-10-06 22:20 UTC by Jilles Tjoelker
Modified: 2010-10-24 19:20 UTC (History)
0 users

See Also:


Attachments
multimedia-xfce4-parole-disable-browser-plugin.patch (573 bytes, patch)
2010-10-06 22:20 UTC, Jilles Tjoelker
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jilles Tjoelker freebsd_committer freebsd_triage 2010-10-06 22:20:01 UTC
The upstream configure script contains broken code in the detection for
the browser plugin:

BROWSER_PLUGIN_DIR="${BROWSER_PLUGIN_DIR:-"\${libdir}/mozilla/plugins"}"

POSIX does not require this to work and many shells such as the current
FreeBSD /bin/sh as of the time of writing silently assign an incorrect
value. (Some shells such as dash even flag a syntax error.)

I plan to change our sh to do what is expected by the author (assign the
literal string ${libdir}/mozilla/plugins if BROWSER_PLUGIN_DIR is not
set yet). This is both because bash and ksh93 do the same thing and
because it is otherwise much harder to make splitting WORD in
${VAR+WORD} work in a consistent manner (the change enforces that each
substitution ends with the same double-quote state as it started).

This change, however, causes the browser plugin to be built, which
subsequently causes a pointyhat failure because the browser plugin's
files are not in the plist. To fix this, I suggest explicitly disabling
the browser plugin via --disable-browser-plugin.

Upstream has removed the browser plugin entirely in their 0.2.2 branch,
although there is no release yet with this change. I would like to
commit the sh changes before that release.

Fix: Apply this patch.
How-To-Repeat: Build/install the port with patched sh.

The patch (to head r212508) is available at
http://www.stack.nl/~jilles/unix/sh-expand-parser-20100919.patch
until I commit it to 9-CURRENT.
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2010-10-06 22:20:13 UTC
Responsible Changed
From-To: freebsd-ports-bugs->miwi

Over to maintainer (via the GNATS Auto Assign Tool)
Comment 2 dfilter service freebsd_committer freebsd_triage 2010-10-24 19:18:55 UTC
johans      2010-10-24 18:18:51 UTC

  FreeBSD ports repository

  Modified files:
    multimedia/xfce4-parole Makefile 
  Log:
  Explicitly disable browser plugin option.
  
  No functional change, as this option is disabled already. But planned
  changes to sh(1) on current might break this implicit behaviour.
  
  PR:             ports/151270
  Submitted by:   jilles
  Approved by:    maintainer timeout (miwi, >14 days)
  
  Revision  Changes    Path
  1.8       +1 -1      ports/multimedia/xfce4-parole/Makefile
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
Comment 3 Johan van Selst freebsd_committer freebsd_triage 2010-10-24 19:18:59 UTC
State Changed
From-To: open->closed

Committed after maintainer timeout.