Bug#1006996: mate-polkit: On arm64 architecture mate-polkit tries to open an amd64 file and fails

2023-03-04 Thread Thomas Uhle

On Fri, 3 Mar 2023, Mike Gabriel wrote:


Hi Thomas,

I'll try to take a look the coming week. Sorry for the delay.

Mike



Thanks!



Bug#1006996: mate-polkit: On arm64 architecture mate-polkit tries to open an amd64 file and fails

2023-03-03 Thread Mike Gabriel

Hi Thomas,

On  Fr 03 Mär 2023 18:55:27 CET, Thomas Uhle wrote:


Dear maintainers,

Mike seems to be quite busy.  I think it would be good if this issue  
could be fixed for Debian bookworm.  So please can anyone of you  
step in?

If there is something else I could do to help, please let me know.

Thank you in advance!

Cheers,

Thomas


I'll try to take a look the coming week. Sorry for the delay.

Mike
--

DAS-NETZWERKTEAM
c\o Technik- und Ökologiezentrum Eckernförde
Mike Gabriel, Marienthaler Str. 17, 24340 Eckernförde
mobile: +49 (1520) 1976 148
landline: +49 (4351) 850 8940

GnuPG Fingerprint: 9BFB AEE8 6C0A A5FF BF22  0782 9AF4 6B30 2577 1B31
mail: mike.gabr...@das-netzwerkteam.de, http://das-netzwerkteam.de



pgpgQeHhmFulL.pgp
Description: Digitale PGP-Signatur


Bug#1006996: mate-polkit: On arm64 architecture mate-polkit tries to open an amd64 file and fails

2023-03-03 Thread Thomas Uhle

Dear maintainers,

Mike seems to be quite busy.  I think it would be good if this issue could 
be fixed for Debian bookworm.  So please can anyone of you step in?

If there is something else I could do to help, please let me know.

Thank you in advance!

Cheers,

Thomas


On Sun, 12 Feb 2023, Thomas Uhle wrote:


Hello Mike,

it seems that you are short of time.  That is why I have updated my patch 
adopting Simon's proposal.  I hope it helps to get this issue fixed for 
Debian bookworm.


Thank you in advance!

Cheers,

Thomas


On Fri, 2 Dec 2022, Mike Gabriel wrote:

> On  Fr 02 Dez 2022 20:55:32 CET, Thomas Uhle wrote:
> 
> > I think that moving polkit-mate-authentication-agent-1 to /usr/libexec and 
> > updating debian/control appropriately would be just enough to close this 
> > ticket.
> 
> Sorry, I am delayed with this. I'll see to an upload the coming days. 
> Thanks to Simon for the various solution approaches, I'll agree with the 
> /usr/libexec switch as the best and least invasive solution.
> 
> Thanks!

> Mike




Bug#1006996: mate-polkit: On arm64 architecture mate-polkit tries to open an amd64 file and fails

2023-02-12 Thread Thomas Uhle

Hello Mike,

it seems that you are short of time.  That is why I have updated my patch 
adopting Simon's proposal.  I hope it helps to get this issue fixed for 
Debian bookworm.


Thank you in advance!

Cheers,

Thomas


On Fri, 2 Dec 2022, Mike Gabriel wrote:


On  Fr 02 Dez 2022 20:55:32 CET, Thomas Uhle wrote:

I think that moving polkit-mate-authentication-agent-1 to /usr/libexec and 
updating debian/control appropriately would be just enough to close this 
ticket.


Sorry, I am delayed with this. I'll see to an upload the coming days. Thanks 
to Simon for the various solution approaches, I'll agree with the 
/usr/libexec switch as the best and least invasive solution.


Thanks!
Mikediff --git a/debian/bin/mate-polkit b/debian/bin/mate-polkit
new file mode 100755
index 000..059a22f
--- /dev/null
+++ b/debian/bin/mate-polkit
@@ -0,0 +1,3 @@
+#!/bin/sh
+
+/usr/libexec/polkit-mate-authentication-agent-1 &
diff --git a/debian/bin/mate-polkit.in b/debian/bin/mate-polkit.in
deleted file mode 100755
index a08dbb5..000
--- a/debian/bin/mate-polkit.in
+++ /dev/null
@@ -1,3 +0,0 @@
-#!/bin/sh
-
-/usr/lib/@DEB_HOST_MULTIARCH@/polkit-mate/polkit-mate-authentication-agent-1 &
diff --git a/debian/control b/debian/control
index e1ccb8c..fe5e461 100644
--- a/debian/control
+++ b/debian/control
@@ -24,9 +24,11 @@
 
 Package: mate-polkit-bin
 Architecture: all
+Multi-Arch: foreign
 Depends: mate-polkit (>= ${source:Version}),
  ${misc:Depends},
-Breaks: mate-polkit (<< 1.12.0-3~),
+Breaks: mate-polkit (<< 1.26.1-2~),
+Replaces: mate-polkit (<< 1.26.1-2~),
 Description: MATE authentication agent for PolicyKit-1 (executable wrapper script)
  The mate-polkit package provides a D-Bus session bus service that is used to
  bring up authentication dialogs used for obtaining privileges.
@@ -40,9 +42,8 @@
 
 Package: mate-polkit
 Architecture: any
-Multi-Arch: same
 Depends: accountsservice,
- mate-polkit-common (= ${binary:Version}),
+ mate-polkit-common (>= ${source:Version}),
  policykit-1,
  ${misc:Depends},
  ${shlibs:Depends},
@@ -54,10 +55,9 @@
  This package contains the MATE policy kit authentication agent.
 
 Package: mate-polkit-common
-Architecture: any
+Architecture: all
 Multi-Arch: foreign
 Depends: ${misc:Depends},
- ${shlibs:Depends},
 Breaks: mate-polkit (<< 1.8.0),
 Description: MATE authentication agent for PolicyKit-1 (common files)
  The mate-polkit package provides a D-Bus session bus service that is used to
diff --git a/debian/mate-polkit-bin.manpages b/debian/mate-polkit-bin.manpages
new file mode 100644
index 000..43f8a64
--- /dev/null
+++ b/debian/mate-polkit-bin.manpages
@@ -0,0 +1 @@
+debian/man/mate-polkit.1
diff --git a/debian/mate-polkit.install b/debian/mate-polkit.install
index b0bfa1a..f1c4c0a 100644
--- a/debian/mate-polkit.install
+++ b/debian/mate-polkit.install
@@ -1 +1 @@
-usr/lib/*/polkit-mate/
+usr/libexec/
diff --git a/debian/mate-polkit.manpages b/debian/mate-polkit.manpages
deleted file mode 100644
index 43f8a64..000
--- a/debian/mate-polkit.manpages
+++ /dev/null
@@ -1 +0,0 @@
-debian/man/mate-polkit.1
diff --git a/debian/rules b/debian/rules
index bceb5d7..a8c7469 100755
--- a/debian/rules
+++ b/debian/rules
@@ -17,11 +17,10 @@ DEB_INSTALL_DOCS_ALL = README AUTHORS HACKING
 
 DEB_CONFIGURE_SCRIPT := ./autogen.sh
 
-DEB_CONFIGURE_EXTRA_FLAGS := --disable-silent-rules \
- --localstatedir=/var \
+DEB_CONFIGURE_EXTRA_FLAGS := --localstatedir=/var \
  --libdir=/usr/lib/$(DEB_HOST_MULTIARCH) \
  --includedir=/usr/include/$(DEB_HOST_MULTIARCH) \
- --libexecdir=/usr/lib/$(DEB_HOST_MULTIARCH)/polkit-mate \
+ --libexecdir=/usr/libexec \
  --disable-silent-rules \
  --disable-static \
  --enable-gtk-doc
@@ -35,7 +34,7 @@ remove-cruft::
 
 install-wrapper::
 	mkdir -p debian/tmp/usr/bin/
-	sed -e 's/@DEB_HOST_MULTIARCH@/$(DEB_HOST_MULTIARCH)/' < debian/bin/mate-polkit.in > debian/tmp/usr/bin/mate-polkit
+	cp -p debian/bin/mate-polkit debian/tmp/usr/bin/mate-polkit
 
 get-orig-source:
 	uscan --noconf --force-download --rename --download-current-version --destdir=..


Bug#1006996: mate-polkit: On arm64 architecture mate-polkit tries to open an amd64 file and fails

2022-12-02 Thread Mike Gabriel

On  Fr 02 Dez 2022 20:55:32 CET, Thomas Uhle wrote:

I think that moving polkit-mate-authentication-agent-1 to  
/usr/libexec and updating debian/control appropriately would be just  
enough to close this ticket.


Sorry, I am delayed with this. I'll see to an upload the coming days.  
Thanks to Simon for the various solution approaches, I'll agree with  
the /usr/libexec switch as the best and least invasive solution.


Thanks!
Mike
--

DAS-NETZWERKTEAM
c\o Technik- und Ökologiezentrum Eckernförde
Mike Gabriel, Marienthaler Str. 17, 24340 Eckernförde
mobile: +49 (1520) 1976 148
landline: +49 (4351) 850 8940

GnuPG Fingerprint: 9BFB AEE8 6C0A A5FF BF22  0782 9AF4 6B30 2577 1B31
mail: mike.gabr...@das-netzwerkteam.de, http://das-netzwerkteam.de



pgpC_9qzEYgKP.pgp
Description: Digitale PGP-Signatur


Bug#1006996: mate-polkit: On arm64 architecture mate-polkit tries to open an amd64 file and fails

2022-12-02 Thread Thomas Uhle

On Thu, 1 Dec 2022, Simon McVittie wrote:


Forwarding this from -devel since I'm not sure whether you saw it there.

[...]

On Fri, 18 Nov 2022 at 22:23:05 +, Mike Gabriel wrote:
> The flaw in mate-polkit is that the
> /etc/xdg/autostart/.desktop file so far has been shipped in
> mate-polkit-common (which usually got built on amd64 builders) and that that
> .desktop file contains a multi-arch path in the Exec= key.

Probably a silly question, but could you build mate-polkit with
--libdir=/usr/libexec instead of /usr/lib/x86_64-linux-gnu/polkit-mate,
so that the polkit auth agent is at a location that does not vary between
architectures?

mate-polkit is currently Multi-Arch: same, but that seems wrong to me:
on a dual-architecture amd64/i386 system (or arm64/armhf or any other
combination), there is no point in running more than one architecture's
copy of the MATE polkit authentication agent. They'll all have the same
UI and the same behaviour, so any single architecture is enough. This
seems like a textbook case of a Multi-Arch: foreign package.


I do agree with Simon.  It really seems to make sense to use /usr/libexec 
as location (as a quick solution at least).  polkit-agent-helper-1 from 
the package policykit-1 for instance is also there.




(On amd64 it would maybe be nice to leave behind a symlink
/usr/lib/x86_64-linux-gnu/polkit-mate/polkit-mate-authentication-agent-1 ->
/usr/libexec/polkit-mate-authentication-agent-1 so that if a sysadmin has
edited the conffile and therefore doesn't receive the new version, then
the agent will still start. But that's an optional quality-of-implementation
thing.)


It's the other way around.  Nobody needs to change a configuration file on 
an amd64 system because this is the only architecture where that path is 
correct.  Anyway, I guess the user would be asked by dpkg during upgrade 
whether to install the new configuration file if it is kept in 
mate-polkit-common and there are already local changes.  So I am not quite 
sure if that additional symlink is really needed.




Another possible solution to this would be to not install an XDG
autostart file into /etc at all, and instead do one or more of these:

1. teach MATE to read additional autostart files from a location below
   /usr as a lower priority than XDG_CONFIG_DIRS, for instance if it
   is currently reading
   [g_get_user_config_dir() + "/autostart",
x + "/autostart" for x in g_get_system_config_dirs().split(":")]
   then that could be changed to
   [g_get_user_config_dir() + "/autostart",
x + "/autostart" for x in g_get_system_config_dirs().split(":"),
"/usr/share/mate/autostart"]
2. launch this polkit agent programmatically in MATE code (perhaps via
   D-Bus activation) instead of relying on XDG autostart as a way to
   discover it
3. integrate it into the desktop shell so it doesn't need to be launched
   separately at all

[...]


I like the second proposal quite much.  It would be good if 
mate-session-manager could take care of it.


Although I fully understand the benefits of the third proposal, I doubt 
that the MATE developers will go this way.  Some of the MATE components 
are also used in X11 environments that are not MATE (e.g. awesome or i3wm) 
because these components are still kind of stand-alone and not bundled 
like their counterparts in GNOME.


However, I guess that this is something that should rather be discussed 
and dealt with upstream.
I think that moving polkit-mate-authentication-agent-1 to /usr/libexec 
and updating debian/control appropriately would be just enough to close 
this ticket.


Cheers,

Thomas



Bug#1006996: Bug #1006996: mate-polkit: On arm64 architecture mate-polkit tries to open an amd64 file and fails

2022-12-01 Thread Simon McVittie
Forwarding this from -devel since I'm not sure whether you saw it there.

- Forwarded message from Simon McVittie  -

Date: Sat, 19 Nov 2022 11:44:34 +
From: Simon McVittie 
To: debian-de...@lists.debian.org
Subject: Re: Bug #1006996: mate-polkit: On arm64 architecture mate-polkit tries 
to open an amd64 file and fails
Message-ID: 

On Fri, 18 Nov 2022 at 22:23:05 +, Mike Gabriel wrote:
> The flaw in mate-polkit is that the
> /etc/xdg/autostart/.desktop file so far has been shipped in
> mate-polkit-common (which usually got built on amd64 builders) and that that
> .desktop file contains a multi-arch path in the Exec= key.

Probably a silly question, but could you build mate-polkit with
--libdir=/usr/libexec instead of /usr/lib/x86_64-linux-gnu/polkit-mate,
so that the polkit auth agent is at a location that does not vary between
architectures?

mate-polkit is currently Multi-Arch: same, but that seems wrong to me:
on a dual-architecture amd64/i386 system (or arm64/armhf or any other
combination), there is no point in running more than one architecture's
copy of the MATE polkit authentication agent. They'll all have the same
UI and the same behaviour, so any single architecture is enough. This
seems like a textbook case of a Multi-Arch: foreign package.

(On amd64 it would maybe be nice to leave behind a symlink
/usr/lib/x86_64-linux-gnu/polkit-mate/polkit-mate-authentication-agent-1 ->
/usr/libexec/polkit-mate-authentication-agent-1 so that if a sysadmin has
edited the conffile and therefore doesn't receive the new version, then
the agent will still start. But that's an optional quality-of-implementation
thing.)

Another possible solution to this would be to not install an XDG
autostart file into /etc at all, and instead do one or more of these:

1. teach MATE to read additional autostart files from a location below
   /usr as a lower priority than XDG_CONFIG_DIRS, for instance if it
   is currently reading
   [g_get_user_config_dir() + "/autostart",
x + "/autostart" for x in g_get_system_config_dirs().split(":")]
   then that could be changed to
   [g_get_user_config_dir() + "/autostart",
x + "/autostart" for x in g_get_system_config_dirs().split(":"),
"/usr/share/mate/autostart"]
2. launch this polkit agent programmatically in MATE code (perhaps via
   D-Bus activation) instead of relying on XDG autostart as a way to
   discover it
3. integrate it into the desktop shell so it doesn't need to be launched
   separately at all

People who have been thinking about image-based OSs are already trying to
move the community towards a model where the "factory-installed" state
of /etc is empty, and every file in /etc represents a local divergence
from that original state (sysadmin configuration). I think that's a
good goal in general: /etc has traditionally been a mixture of sysadmin
configuration (/etc/fstab, /etc/hostname) and system-integration glue
(/etc/ld.so.conf.d, /etc/xdg/autostart), and if we separated those two,
it would become a lot more obvious what is sysadmin configuration and
what is part of the OS.

Solving the problem of "the design of /etc/xdg/autostart conflicts with
wanting a factory install to have an empty /etc" *in general* is hard
because it's an interoperability point between multiple desktop environments
(e.g. people don't want to break /etc/xdg/aa-notify.desktop), so changing
it in an interoperable/consistent way requires changing the spec and every
implementation. For instance some people are advocating changing the
autostart spec so that implementations will read
/usr/etc/xdg/autostart/*.desktop or /usr/etc/autostart/*.desktop or
/usr/share/xdg/autostart/*.desktop or some standardized path like that.
AIUI there's currently no real consensus on specific paths.

However, solving it for a component that has OnlyShowIn=MATE is easy,
because by design only MATE is going to launch that component anyway -
so it's entirely reasonable for it to rely on implementation-specific
MATE behaviours. For instance, if MATE developers decided that they are
going to read /usr/share/mate/autostart as an additional search path
component of lower priority than $XDG_CONFIG_DIRS/autostart, then that
wouldn't harm interoperability with other desktops and is something
that MATE can do unilaterally.

Alternatively, integrating the polkit agent into the desktop shell is
what GNOME does: GNOME's polkit agent is part of GNOME Shell, which
has some other advantages, such as the compositor (which is also Shell)
being able to give it special handling around input-grabbing to protect
the secure input path.

If you evaluate those options and decide that you still need
conffile handling for /etc/xdg/autostart/*.desktop here:

> To simplify life (and yes, this is debatable): Do situations exist, where an
> enforced conffile update (overwriting it) is allowed / justifiable?

The Policy requirement is not "don't overwrite conffiles", the Policy
requirement is "don't revert sysadmin changes". 

Bug#1006996: mate-polkit: On arm64 architecture mate-polkit tries to open an amd64 file and fails

2022-11-27 Thread Thomas Uhle

On Fri, 18 Nov 2022, Mike Gabriel wrote:


Hi Thomas, dear DD colleagues (et al.),

[...]

The change-over of the XDG autostart file from common:pkg mate-polkit-common 
to bin:pkg mate-polkit is not as simple as it seems.


[...]



Hello Mike,

you are absolutely right.  I had forgotten to add appropriate Breaks and 
Replaces to debian/control as well as mate-polkit-common.maintscript with 
a corresponding rm_conffile line.  I have updated the patch accordingly 
and assumed that the next version would be '1.26.1-2'.


Cheers,

Thomasdiff --git a/debian/control b/debian/control
index e1ccb8c..59566ae 100644
--- a/debian/control
+++ b/debian/control
@@ -23,10 +23,12 @@
 Vcs-Git: https://salsa.debian.org/debian-mate-team/mate-polkit.git
 
 Package: mate-polkit-bin
-Architecture: all
-Depends: mate-polkit (>= ${source:Version}),
+Architecture: any
+Depends: mate-polkit (= ${binary:Version}),
  ${misc:Depends},
+ ${shlibs:Depends},
-Breaks: mate-polkit (<< 1.12.0-3~),
+Breaks: mate-polkit (<< 1.26.1-2~),
+Replaces: mate-polkit (<< 1.26.1-2~),
 Description: MATE authentication agent for PolicyKit-1 (executable wrapper script)
  The mate-polkit package provides a D-Bus session bus service that is used to
  bring up authentication dialogs used for obtaining privileges.
@@ -40,13 +42,14 @@
 
 Package: mate-polkit
 Architecture: any
-Multi-Arch: same
 Depends: accountsservice,
- mate-polkit-common (= ${binary:Version}),
+ mate-polkit-common (>= ${source:Version}),
  policykit-1,
  ${misc:Depends},
  ${shlibs:Depends},
 Provides: polkit-1-auth-agent,
+Breaks: mate-polkit-common (<< 1.26.1-2~),
+Replaces: mate-polkit-common (<< 1.26.1-2~),
 Description: MATE authentication agent for PolicyKit-1
  The mate-polkit package provides a D-Bus session bus service that is used to
  bring up authentication dialogs used for obtaining privileges.
@@ -54,10 +57,9 @@
  This package contains the MATE policy kit authentication agent.
 
 Package: mate-polkit-common
-Architecture: any
+Architecture: all
 Multi-Arch: foreign
 Depends: ${misc:Depends},
- ${shlibs:Depends},
 Breaks: mate-polkit (<< 1.8.0),
 Description: MATE authentication agent for PolicyKit-1 (common files)
  The mate-polkit package provides a D-Bus session bus service that is used to
diff --git a/debian/mate-polkit-bin.manpages b/debian/mate-polkit-bin.manpages
new file mode 100644
index 000..43f8a64
--- /dev/null
+++ b/debian/mate-polkit-bin.manpages
@@ -0,0 +1 @@
+debian/man/mate-polkit.1
diff --git a/debian/mate-polkit-common.install b/debian/mate-polkit-common.install
index 7034e54..2568e1f 100644
--- a/debian/mate-polkit-common.install
+++ b/debian/mate-polkit-common.install
@@ -1,2 +1 @@
-etc/
 usr/share/locale/
diff --git a/debian/mate-polkit-common.maintscript b/debian/mate-polkit-common.maintscript
new file mode 100644
index 000..e43ea94
--- /dev/null
+++ b/debian/mate-polkit-common.maintscript
@@ -0,0 +1 @@
+rm_conffile /etc/xdg/autostart/polkit-mate-authentication-agent-1.desktop 1.26.1-2~
diff --git a/debian/mate-polkit.install b/debian/mate-polkit.install
index b0bfa1a..3d97aa9 100644
--- a/debian/mate-polkit.install
+++ b/debian/mate-polkit.install
@@ -1 +1,2 @@
+etc/
 usr/lib/*/polkit-mate/
diff --git a/debian/mate-polkit.manpages b/debian/mate-polkit.manpages
deleted file mode 100644
index 43f8a64..000
--- a/debian/mate-polkit.manpages
+++ /dev/null
@@ -1 +0,0 @@
-debian/man/mate-polkit.1


Bug#1006996: Bug #1006996: mate-polkit: On arm64 architecture mate-polkit tries to open an amd64 file and fails

2022-11-18 Thread Mike Gabriel

Hi Thomas, dear DD colleagues (et al.),

On  Fr 18 Nov 2022 20:55:39 CET, Thomas Uhle wrote:


Dear maintainers,

in addition to Christian Britz' report, the autostart desktop file  
should also be in mate-polkit because of the architecture triplet in  
the path to polkit-mate-authentication-agent-1, and the provided  
manpage has not yet been moved to mate-polkit-bin what causes a  
lintian issue.


I have prepared a patch that should fix all these packaging issues.

Best regards,

Thomas Uhle


The change-over of the XDG autostart file from common:pkg  
mate-polkit-common to bin:pkg mate-polkit is not as simple as it seems.


This is why I am cross-posting to the debian-devel mailing list and  
would like to ask for ideas...


Here is the challenge:

The flaw in mate-polkit is that the  
/etc/xdg/autostart/.desktop file so far has been shipped  
in mate-polkit-common (which usually got built on amd64 builders) and  
that that .desktop file contains a multi-arch path in the Exec= key.


So, for non-amd64 architectures, the .desktop files tries to execute a  
polkit-agent binary at some amd64 multi-arch path. Thanks to Christian  
and Thomas for dealing with this and reporting the issue.


With the next upload of mate-polkit src:pkg I want to achieve these things:

  * move /etc/xdg/autostart/.desktop from  
mate-polkit-common (arch:all) to

mate-polkit (arch:any), ... and yes, I have seen #595112

  * as the Exec= field in this file definitely needs to get updated  
on non-amd64 machines

(see #1006996), I most certainly don't want to keep versions of that
file (stemming from arch-indep mate-polkit-common) that got  
modified by the site/machine
admin. In most cases, I want to install the new version from the  
arch-dep package mate-polkit.


  * so for non-amd64 architectures, the best solution probably is to  
remove the
previous .desktop (mate-polkit-common) file and  
replace it by the new

.desktop file (ignoring local changes)

Question to DD colleagues: How can this be achieve in a clean way  
without piuparts (or other QA tools) yelling at me afterwards? Any  
spontaneous ideas???


To simplify life (and yes, this is debatable): Do situations exist,  
where an enforced conffile update (overwriting it) is allowed /  
justifiable? If so, do you know example packages that exercise this?


Thanks in advance for any feedback!
Mike




--

DAS-NETZWERKTEAM
c\o Technik- und Ökologiezentrum Eckernförde
Mike Gabriel, Marienthaler Str. 17, 24340 Eckernförde
mobile: +49 (1520) 1976 148
landline: +49 (4351) 850 8940

GnuPG Fingerprint: 9BFB AEE8 6C0A A5FF BF22  0782 9AF4 6B30 2577 1B31
mail: mike.gabr...@das-netzwerkteam.de, http://das-netzwerkteam.de



pgp3xKzepzJj6.pgp
Description: Digitale PGP-Signatur


Bug#1006996: Bug #1006996: mate-polkit: On arm64 architecture mate-polkit tries to open an amd64 file and fails

2022-11-18 Thread Thomas Uhle

Control: affects -1 mate-polkit-bin
Control: tags -1 patch

Dear maintainers,

in addition to Christian Britz' report, the autostart desktop file should 
also be in mate-polkit because of the architecture triplet in the path to 
polkit-mate-authentication-agent-1, and the provided manpage has not yet 
been moved to mate-polkit-bin what causes a lintian issue.


I have prepared a patch that should fix all these packaging issues.

Best regards,

Thomas Uhle



On Thu, 10 Mar 2022, Christian Britz wrote:


Package: mate-polkit
Version: 1.24.0-2
Severity: important
X-Debbugs-Cc: cbr...@t-online.de

Dear Maintainer,

on arm64 architecture, mate-polkit tries to open an amd64 file and fails:

mate-polkit &
[1] 25011
cbritz@raspberrypi:~$ /usr/bin/mate-polkit: 3: 
/usr/lib/x86_64-linux-gnu/polkit-mate/polkit-mate-authentication-agent-1: not 
found

This renders the package unusable on arm64.

Regards,
Christian

-- System Information:
Debian Release: 11.2
 APT prefers stable-updates
 APT policy: (500, 'stable-updates'), (500, 'stable-security'), (500, 'stable')
Architecture: arm64 (aarch64)

Kernel: Linux 5.10.0-12-arm64 (SMP w/4 CPU threads)
Kernel taint flags: TAINT_CRAP
Locale: LANG=de_DE.UTF-8, LC_CTYPE=de_DE.UTF-8 (charmap=UTF-8), LANGUAGE not set
Shell: /bin/sh linked to /usr/bin/dash
Init: systemd (via /run/systemd/system)
LSM: AppArmor: enabled

Versions of packages mate-polkit depends on:
ii  accountsservice0.6.55-3
ii  libc6  2.31-13+deb11u2
ii  libgdk-pixbuf2.0-0 2.40.2-2
ii  libglib2.0-0   2.66.8-1
ii  libgtk-3-0 3.24.24-4
ii  libpolkit-agent-1-00.105-31+deb11u1
ii  libpolkit-gobject-1-0  0.105-31+deb11u1
ii  mate-polkit-common 1.24.0-2
ii  policykit-10.105-31+deb11u1

mate-polkit recommends no packages.

mate-polkit suggests no packages.
diff --git a/debian/control b/debian/control
index e1ccb8c..990d5cc 100644
--- a/debian/control
+++ b/debian/control
@@ -23,10 +23,10 @@
 Vcs-Git: https://salsa.debian.org/debian-mate-team/mate-polkit.git
 
 Package: mate-polkit-bin
-Architecture: all
-Depends: mate-polkit (>= ${source:Version}),
+Architecture: any
+Depends: mate-polkit (= ${binary:Version}),
  ${misc:Depends},
-Breaks: mate-polkit (<< 1.12.0-3~),
+ ${shlibs:Depends},
 Description: MATE authentication agent for PolicyKit-1 (executable wrapper script)
  The mate-polkit package provides a D-Bus session bus service that is used to
  bring up authentication dialogs used for obtaining privileges.
@@ -40,9 +40,8 @@
 
 Package: mate-polkit
 Architecture: any
-Multi-Arch: same
 Depends: accountsservice,
- mate-polkit-common (= ${binary:Version}),
+ mate-polkit-common (>= ${source:Version}),
  policykit-1,
  ${misc:Depends},
  ${shlibs:Depends},
@@ -54,10 +53,9 @@
  This package contains the MATE policy kit authentication agent.
 
 Package: mate-polkit-common
-Architecture: any
+Architecture: all
 Multi-Arch: foreign
 Depends: ${misc:Depends},
- ${shlibs:Depends},
 Breaks: mate-polkit (<< 1.8.0),
 Description: MATE authentication agent for PolicyKit-1 (common files)
  The mate-polkit package provides a D-Bus session bus service that is used to
diff --git a/debian/mate-polkit.install b/debian/mate-polkit.install
index b0bfa1a..3d97aa9 100644
--- a/debian/mate-polkit.install
+++ b/debian/mate-polkit.install
@@ -1 +1,2 @@
+etc/
 usr/lib/*/polkit-mate/
diff --git a/debian/mate-polkit-common.install b/debian/mate-polkit-common.install
index 7034e54..2568e1f 100644
--- a/debian/mate-polkit-common.install
+++ b/debian/mate-polkit-common.install
@@ -1,2 +1 @@
-etc/
 usr/share/locale/
diff --git a/debian/mate-polkit.manpages b/debian/mate-polkit.manpages
deleted file mode 100644
index 43f8a64..000
--- a/debian/mate-polkit.manpages
+++ /dev/null
@@ -1 +0,0 @@
-debian/man/mate-polkit.1
diff --git a/debian/mate-polkit-bin.manpages b/debian/mate-polkit-bin.manpages
new file mode 100644
index 000..43f8a64
--- /dev/null
+++ b/debian/mate-polkit-bin.manpages
@@ -0,0 +1 @@
+debian/man/mate-polkit.1


Bug#1006996: mate-polkit: On arm64 architecture mate-polkit tries to open an amd64 file and fails

2022-03-10 Thread Christian Britz
Package: mate-polkit
Version: 1.24.0-2
Severity: important
X-Debbugs-Cc: cbr...@t-online.de

Dear Maintainer,

on arm64 architecture, mate-polkit tries to open an amd64 file and fails:

mate-polkit &
[1] 25011
cbritz@raspberrypi:~$ /usr/bin/mate-polkit: 3: /usr/lib/x86_64-linux-
gnu/polkit-mate/polkit-mate-authentication-agent-1: not found

This renders the package unusable on arm64.

Regards,
Christian

-- System Information:
Debian Release: 11.2
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'stable-security'), (500,
'stable')
Architecture: arm64 (aarch64)

Kernel: Linux 5.10.0-12-arm64 (SMP w/4 CPU threads)
Kernel taint flags: TAINT_CRAP
Locale: LANG=de_DE.UTF-8, LC_CTYPE=de_DE.UTF-8 (charmap=UTF-8), LANGUAGE not
set
Shell: /bin/sh linked to /usr/bin/dash
Init: systemd (via /run/systemd/system)
LSM: AppArmor: enabled

Versions of packages mate-polkit depends on:
ii  accountsservice0.6.55-3
ii  libc6  2.31-13+deb11u2
ii  libgdk-pixbuf2.0-0 2.40.2-2
ii  libglib2.0-0   2.66.8-1
ii  libgtk-3-0 3.24.24-4
ii  libpolkit-agent-1-00.105-31+deb11u1
ii  libpolkit-gobject-1-0  0.105-31+deb11u1
ii  mate-polkit-common 1.24.0-2
ii  policykit-10.105-31+deb11u1

mate-polkit recommends no packages.

mate-polkit suggests no packages.