[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#923322: plasma-browser-integration: Incorrect installation directory for chrome config file



Hi David and all,

On Tue, Feb 26, 2019 at 12:50:43PM +0000, David Edmundson wrote:
> Plasma browser integration needs to install a file to /etc/opt
>
> Upstream does this.
>
> There was a concious decision in debian packaging to do something else,
> but the move to break it cites a rule about installing into /opt
> This does not apply as it refers to a completely different directory.
> Note the /etc prefix.

It was brought to my attention that the Plasma Integration extension for
Chrome [1] has this note in its description:

  NOTE: This extension is not supported on Debian.

As far as I understand, this extension works fine with Chromium from
our repository, but does not work (out of the box) with Google Chrome
when it is installed from Google's repository.

On one hand, Debian's packages are not obliged to be compatible with
third-party packages. On the other hand, people who use Google's builds
may blame Debian if things are not working for them.

> A solution has been found for chrome-gnome-shell using postinst/postrm 
> to copy the file into the correct location.
>
> Can we have the same system used here please.

That system is a bit more complex than it sounds, as the postrm script
actually needs to *recreate* /etc/opt directory after dpkg deletes it.
See the bug [2] for details.

The good news is that chrome-gnome-shell maintainers have already done
the hard work, so we can just copy their logic (with the json file name
changed, obviously) if we decide to go that way.

Maxy, Scarlett, what do you think about this?

I do not think we should fix this for Buster, as the fix would be quite
tricky and if I were the release team I would not like it :)

Also, users who are affected may follow the instruction that we ship in
/usr/share/doc/plasma-browser-integration/README.Debian [1] and add the
symlink manually. I wonder if maybe the extension description can mention
that instruction (instead of just saying Debian is not supported)?

[1]: https://chrome.google.com/webstore/detail/plasma-integration/cimiefiiaegbelhefglklhhakcgmhkai
[2]: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=888549
[3]: https://salsa.debian.org/qt-kde-team/kde/plasma-browser-integration/blob/master/debian/README.Debian

--
Dmitry Shachnev

Attachment: signature.asc
Description: PGP signature


Reply to: