Oh I even thought we did a bigger oopsie here of a component mismatch (nmap in 
main but menu is not). Yet I found that zenmap (binary) of src:nmap is not in 
main either which explains why it migrated.
Maybe I should not have done the review in between meeting :-/ as also the bug 
reference you added was not following the regexp, but I set the status 
correctly here.

It is ok'ish and working for now, but we should follow on to clean up.

Per discussion in the linked Debian bug policykit-1 was recommended
which is better (This was to desktoppy for me to know right away and I
was just following the command that was used so far in my suggestion).

About recommends, the package (strictly speaking) can work without this,
you can sudo it or use a root X11 session. So it really is more a
recommends than a depends.

The calls look slightly different, but we need to adapt the su-to-root patch 
anyway.
Other examples of pkexec in .desktop files used wrappers, but those only do 
like:
pkexec "/usr/sbin/gparted" "$@"
Well we can do that directly in the desktop file I think unless somebody 
explains why a wrapper is needed. Well actually to run it from cmdline it would 
be nice to have one.

I had some issues that zenmap can't access the Display in that environment, but 
other programs could using the same mechanisms. I assume you need a bit more 
for polkit to work as it should in this case. The manpage mentions that it 
intentionally strips the environment.
After knowing that I found [1]
This is such a case as zenmap has no own polkit integration to do it right - so 
it is one of the legacy apps that needs this , see also [2] in regard to X11.

So we also need a policykit action *sigh*.

In a few iterations I got closer bug I think I'm not complete.
I'll attach a patch not meant to be complete yet (it might be incomplete and 
have side effects due to neither of us being a Desktop/policykit expert).

@David - if you'd give this another look to have a second set of eyes on
it and finalize it to work that would be great. If you are good with it
(or even with a partial suggest) you might pass it to the Debian bug. If
you don't find it either maybe Rafael has the experience and do the
cleanup of this even if we don't know all of the bits it needs.

[1]: 
https://unix.stackexchange.com/questions/203136/how-do-i-run-gui-applications-as-root-by-using-pkexec
 
[2]: http://manpages.ubuntu.com/manpages/xenial/man1/pkexec.1.html

** Patch added: "debdiff of experimental rework to use polkit/pkexec - (WIP)"
   
https://bugs.launchpad.net/ubuntu/+source/nmap/+bug/1263311/+attachment/5059474/+files/nmap-zenmap-use-pkexec-experimental.debdiff

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1263311

Title:
  zenmap as root not displayed

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/nmap/+bug/1263311/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to