Re: [gentoo-proxy-maint] [PATCH] www-plugins/passff-host: Fix for firefox-bin with no-symlink-lib

[Haelwenn (lanodan) Monnier]

[2019-06-28 12:13:23+0200] Michał Górny:
> On Fri, 2019-06-28 at 12:09 +0200, Haelwenn (lanodan) Monnier wrote:
> > [2019-06-28 10:31:51+0200] Michał Górny:
> > > On Fri, 2019-06-28 at 08:48 +0200, Haelwenn (lanodan) Monnier wrote:
> > > > Closes: https://bugs.gentoo.org/show_bug.cgi?id=687746
> > > > Signed-off-by: Haelwenn (lanodan) Monnier <contact@×××××××××.me>
> > > > ---
> > > >  ...sff-host-1.2.0.ebuild => passff-host-1.2.0-r1.ebuild} | 9 ++++++++-
> > > >  1 file changed, 8 insertions(+), 1 deletion(-)
> > > >  rename www-plugins/passff-host/{passff-host-1.2.0.ebuild => passff-host-1.2.0-r1.ebuild} (84%)
> > > >
> > > > diff --git a/www-plugins/passff-host/passff-host-1.2.0.ebuild b/www-plugins/passff-host/passff-host-1.2.0-r1.ebuild
> > > > similarity index 84%
> > > > rename from www-plugins/passff-host/passff-host-1.2.0.ebuild
> > > > rename to www-plugins/passff-host/passff-host-1.2.0-r1.ebuild
> > > > index f42fb96ba19..75bebf643b8 100644
> > > > --- a/www-plugins/passff-host/passff-host-1.2.0.ebuild
> > > > +++ b/www-plugins/passff-host/passff-host-1.2.0-r1.ebuild
> > > > @@ -1,52 +1,59 @@
> > > >  # Copyright 2018-2019 Gentoo Authors
> > > >  # Distributed under the terms of the GNU General Public License v2
> > > >
> > > >  EAPI=7
> > > >
> > > >  PYTHON_COMPAT=( python3_5 python3_6 python3_7 )
> > > >
> > > >  inherit python-single-r1
> > > >
> > > >  DESCRIPTION="Host app for the PassFF WebExtension"
> > > >  HOMEPAGE="https://github.com/passff/passff-host"
> > > >
> > > >  SRC_URI="
> > > >  	https://github.com/passff/passff-host/releases/download/${PV}/passff.py -> ${P}.py
> > > >  	https://github.com/passff/passff-host/releases/download/${PV}/passff.json -> ${P}.json
> > > >  "
> > > >
> > > >  LICENSE="GPL-2"
> > > >  SLOT="0"
> > > >  KEYWORDS="~amd64 ~x86"
> > > >  IUSE="chrome chromium firefox vivaldi"
> > > >  REQUIRED_USE="|| ( chrome chromium firefox vivaldi )"
> > > >
> > > >  S="${WORKDIR}"
> > > >
> > > >  src_unpack() {
> > > >  	cp "${DISTDIR}/${P}.json" . || die
> > > >  	cp "${DISTDIR}/${P}.py" . || die
> > > >  }
> > > >
> > > >  src_prepare() {
> > > >  	default
> > > >  	python_fix_shebang "${P}.py"
> > > >  }
> > > >
> > > >  src_install() {
> > > >  	local target_dirs=()
> > > >
> > > >  	use chrome   && target_dirs+=( "/etc/opt/chrome/native-messaging-hosts" )
> > > >  	use chromium && target_dirs+=( "/etc/chromium/native-messaging-hosts" )
> > > > -	use firefox  && target_dirs+=( "/usr/$(get_libdir)/mozilla/native-messaging-hosts" )
> > > > +	if use firefox; then
> > > > +		target_dirs+=( "/usr/$(get_libdir)/mozilla/native-messaging-hosts" )
> > > > +
> > > > +		# firefox-bin compatibility
> > > > +		if [[ "$(get_libdir)" != "lib" ]]; then
> > > > +			target_dirs+=( "/usr/lib/mozilla/native-messaging-hosts" )
> > > > +		fi
> > > > +	fi
> > >
> > > Wouldn't this break the migration?  I think this will cause unsymlink-
> > > lib to detect collision if this package is installed before the switch.
> >
> > Ah, yes it does, tried on a fresh chroot and I get this message when
> > running `unsymlink-lib --analyze`:
> >
> > One or more files are both in /usr/lib/ and /usr/lib64/, making the conversion impossible.
> >
> >         mozilla/native-messaging-hosts/passff.json
> >         mozilla/native-messaging-hosts/passff.py
> >
> > ---
> >
> > So do you have any recommendation for fixing this issue as it needs to
> > be in both /usr/$(get_libdir) for www-client/firefox with
> > SYMLINK_LIB=no and /usr/lib for www-client/firefox-bin.
>
> I think some packages solve this by checking SYMLINK_LIB value
> explicitly.

I would have preferred avoiding that because it would basically need a
manual rebuild after the migration because it doesn't goes in
/{usr/,}lib32 at all.

So I guess I'll pass the bug to firefox-bin maintainers.

> > Otherwise I guess I would file a bug report referencing
> > https://bugs.gentoo.org/show_bug.cgi?id=687746 to get firefox-bin
> > fixed instead or assign the maintainers to the bug.
> >
> > > >  	use vivaldi  && target_dirs+=( "/etc/vivaldi/native-messaging-hosts" )
> > > >
> > > >  	for target_dir in "${target_dirs[@]}"; do
> > > >  		sed "s;PLACEHOLDER;${target_dir}/passff.py;g" "${P}.json" > "passff.json" || die
> > > >
> > > >  		insinto "${target_dir}"
> > > >  		doins passff.json
> > > >  		exeinto "${target_dir}"
> > > >  		newexe "${P}.py" passff.py
> > > >  	done
> > > >  }
> > >
> > > --
> > > Best regards,
> > > Michał Górny
>
> --
> Best regards,
> Michał Górny