On Thu, 4 Feb 2016 15:49:31 +0200 "opensuse.lietuviu.kalba" wrote:

...

On Thu, 4 Feb 2016 15:06:12 +0200 "opensuse.lietuviu.kalba" wrote:

...

Hi,

I created package [1], that has program to open RPM packages in YaST2 software manager (similarly as in /usr/bin/package-manager from libzypp (<=13.9) shipped with openSUSE 13.1 and earlier). Some description is FATE [2]. I have no rights to submit package directly to YaST:Head or Factory. Please, can someone submit it there? Or [better], can someone please integrate this into yast2-packager https://github.com/yast/yast-packager ?

[1] https://build.opensuse.org/package/show/home:embar-:Lietukas/yast2-rpm-handl... [2] https://features.opensuse.org/319376

Hi, what is benefit of such script? I check code and it is basically xdg-su -c /sbin/yast2 sw_single <args>

if it is really needed, then maybe better place is to just define alias in shell? I worry that unless we have clear use case, then I am ( personally, not as suse nor as yast team ) against adding it back toyast.

Josef Use case: Open several local RPM packages (downloaded from internet) in YaST Software manager from File manager (like Dolphin). Today there is no-way to do this; downloaded RPM packages now is opened with Ark archiver in KDE environement, or Gnome Software Center (if installed). Some very new users try to install these packages via `rpm` and fails, because it fails to install dependencies (`zypper` can do

2016.02.04 15:14, Josef Reidinger rašė: this, but new users usually don't know this command)...

Does this script really solve it? what about adding instead only https://build.opensuse.org/package/view_file/home:embar-:Lietukas/yast2-rpm-... to packager and replace exec with something like xdg-su -c "/sbin/yast2 sw_single %F" so it is similar to existing desktop files? https://github.com/yast/yast-packager/tree/master/src/desktop Or maybe even better idea can we extend existing desktop file to handle rpm files and allow passing %F? https://github.com/yast/yast-packager/blob/master/src/desktop/sw_single.desk... Josef -- To unsubscribe, e-mail: yast-devel+unsubscribe@opensuse.org To contact the owner, e-mail: yast-devel+owner@opensuse.org

2016.02.04 16:14, opensuse.lietuviu.kalba rašė:

2016.02.04 15:56, Josef Reidinger rašė:

...

On Thu, 4 Feb 2016 15:49:31 +0200 "opensuse.lietuviu.kalba" wrote:

...

...

On Thu, 4 Feb 2016 15:06:12 +0200 "opensuse.lietuviu.kalba" wrote:

...

Hi,

I created package [1], that has program to open RPM packages in YaST2 software manager (similarly as in /usr/bin/package-manager from libzypp (<=13.9) shipped with openSUSE 13.1 and earlier). Some description is FATE [2]. I have no rights to submit package directly to YaST:Head or Factory. Please, can someone submit it there? Or [better], can someone please integrate this into yast2-packager https://github.com/yast/yast-packager ?

[1] https://build.opensuse.org/package/show/home:embar-:Lietukas/yast2-rpm-handl...

[2] https://features.opensuse.org/319376 Hi, what is benefit of such script? I check code and it is basically xdg-su -c /sbin/yast2 sw_single <args>

if it is really needed, then maybe better place is to just define alias in shell? I worry that unless we have clear use case, then I am ( personally, not as suse nor as yast team ) against adding it back toyast.

Josef Use case: Open several local RPM packages (downloaded from internet) in YaST Software manager from File manager (like Dolphin). Today there is no-way to do this; downloaded RPM packages now is opened with Ark archiver in KDE environement, or Gnome Software Center (if installed). Some very new users try to install these packages via `rpm` and fails, because it fails to install dependencies (`zypper` can do

2016.02.04 15:14, Josef Reidinger rašė: this, but new users usually don't know this command)...

Does this script really solve it? what about adding instead only https://build.opensuse.org/package/view_file/home:embar-:Lietukas/yast2-rpm-...

to packager and replace exec with something like xdg-su -c "/sbin/yast2 sw_single %F"

so it is similar to existing desktop files? https://github.com/yast/yast-packager/tree/master/src/desktop

Or maybe even better idea can we extend existing desktop file to handle rpm files and allow passing %F?

https://github.com/yast/yast-packager/blob/master/src/desktop/sw_single.desk...

Josef Yes, script solves this. Yes, I agree, better solution is to edit https://github.com/yast/yast-packager/blob/master/src/desktop/sw_single.desk...

by adding xdg-su -c "/sbin/yast2 sw_single %F" MimeType=application/x-rpm;

But `xdg-su -c "/sbin/yast2 sw_single %F"` will fail if some directory or filename has space, but script converts special symbols by adding \ before each special symbol So script is superior, if there are spaces in path of files! -- To unsubscribe, e-mail: yast-devel+unsubscribe@opensuse.org To contact the owner, e-mail: yast-devel+owner@opensuse.org

On Thu, 4 Feb 2016 15:36:59 +0100 Josef Reidinger wrote:

On Thu, 4 Feb 2016 16:22:18 +0200 "opensuse.lietuviu.kalba" wrote:

...

2016.02.04 16:14, opensuse.lietuviu.kalba rašė:

...

2016.02.04 15:56, Josef Reidinger rašė:

...

On Thu, 4 Feb 2016 15:49:31 +0200 "opensuse.lietuviu.kalba" wrote:

...

...

On Thu, 4 Feb 2016 15:06:12 +0200 "opensuse.lietuviu.kalba" wrote: > Hi, > > I created package [1], that has program to open RPM packages > in YaST2 software manager (similarly as > in /usr/bin/package-manager from libzypp (<=13.9) shipped > with openSUSE 13.1 and earlier). Some description is FATE [2]. > I have no rights to submit package directly to YaST:Head or > Factory. Please, can someone submit it there? > Or [better], can someone please integrate this into > yast2-packager https://github.com/yast/yast-packager ? > > [1] > https://build.opensuse.org/package/show/home:embar-:Lietukas/yast2-rpm-handl... > > [2] https://features.opensuse.org/319376 Hi, what is benefit of such script? I check code and it is basically xdg-su -c /sbin/yast2 sw_single <args>

if it is really needed, then maybe better place is to just define alias in shell? I worry that unless we have clear use case, then I am ( personally, not as suse nor as yast team ) against adding it back toyast.

Josef Use case: Open several local RPM packages (downloaded from internet) in YaST Software manager from File manager (like Dolphin). Today there is no-way to do this; downloaded RPM packages now is opened with Ark archiver in KDE environement, or Gnome Software Center (if installed). Some very new users try to install these packages via `rpm` and fails, because it fails to install dependencies (`zypper` can do

2016.02.04 15:14, Josef Reidinger rašė: this, but new users usually don't know this command)...

Does this script really solve it? what about adding instead only https://build.opensuse.org/package/view_file/home:embar-:Lietukas/yast2-rpm-...

to packager and replace exec with something like xdg-su -c "/sbin/yast2 sw_single %F"

so it is similar to existing desktop files? https://github.com/yast/yast-packager/tree/master/src/desktop

Or maybe even better idea can we extend existing desktop file to handle rpm files and allow passing %F?

https://github.com/yast/yast-packager/blob/master/src/desktop/sw_single.desk...

Josef Yes, script solves this. Yes, I agree, better solution is to edit https://github.com/yast/yast-packager/blob/master/src/desktop/sw_single.desk...

by adding xdg-su -c "/sbin/yast2 sw_single %F" MimeType=application/x-rpm;

But `xdg-su -c "/sbin/yast2 sw_single %F"` will fail if some directory or filename has space, but script converts special symbols by adding \ before each special symbol So script is superior, if there are spaces in path of files!

So my question is how other desktop files do it? I do not believe that each desktop file have command line wrapper, that handle special characters. I propose to use common solution rather then creating own one that we have to maintain.

I will answer myself, that they just passed %F and according to specification http://standards.freedesktop.org/desktop-entry-spec/desktop-entry-spec-lates... the %F is A list of files. Use for apps that can open several local files at once. Each file is passed as a separate argument to the executable program. So if it is not escaped, it is for me problem of program that interpret desktop files itself as example on that page also use just %F http://standards.freedesktop.org/desktop-entry-spec/desktop-entry-spec-lates... As Martin Vidner, author of script is on this mailing list I will let him to also comment it. Josef -- To unsubscribe, e-mail: yast-devel+unsubscribe@opensuse.org To contact the owner, e-mail: yast-devel+owner@opensuse.org

On Thu, 4 Feb 2016 17:01:04 +0200 "opensuse.lietuviu.kalba" wrote:

2016.02.04 16:45, Josef Reidinger rašė:

...

On Thu, 4 Feb 2016 15:36:59 +0100 Josef Reidinger wrote:

...

So my question is how other desktop files do it? I do not believe that each desktop file have command line wrapper, that handle special characters. I propose to use common solution rather then creating own one that we have to maintain.

I will answer myself, that they just passed %F and according to specification http://standards.freedesktop.org/desktop-entry-spec/desktop-entry-spec-lates... the %F is A list of files. Use for apps that can open several local files at once. Each file is passed as a separate argument to the executable program.

So if it is not escaped, it is for me problem of program that interpret desktop files itself as example on that page also use just %F http://standards.freedesktop.org/desktop-entry-spec/desktop-entry-spec-lates...

As Martin Vidner, author of script is on this mailing list I will let him to also comment it.

Josef

So yes, I totally agree, that perfect solution is 1) fix `/sbin/yast2 sw_single` to accept multiple files with spaces as arguments 2) also add in https://github.com/yast/yast-packager/blob/master/src/desktop/sw_single.desk...

xdg-su -c "/sbin/yast2 sw_single %F" MimeType=application/x-rpm;

Hmm, it is actually there: https://github.com/yast/yast-packager/blob/master/desktop/yast2-packager.des... Problem looks like line NotShowIn=KDE;GNOME;MATE; I will investigate it tomorrow. -- To unsubscribe, e-mail: yast-devel+unsubscribe@opensuse.org To contact the owner, e-mail: yast-devel+owner@opensuse.org

On Thu, 4 Feb 2016 20:22:57 +0200 "opensuse.lietuviu.kalba" wrote:

2016.02.04 17:10, Josef Reidinger rašė:

...

On Thu, 4 Feb 2016 17:01:04 +0200 "opensuse.lietuviu.kalba" wrote:

...

So yes, I totally agree, that perfect solution is 1) fix `/sbin/yast2 sw_single` to accept multiple files with spaces as arguments 2) also add in https://github.com/yast/yast-packager/blob/master/src/desktop/sw_single.desk...

xdg-su -c "/sbin/yast2 sw_single %F" MimeType=application/x-rpm;

Hmm, it is actually there: https://github.com/yast/yast-packager/blob/master/desktop/yast2-packager.des...

Problem looks like line NotShowIn=KDE;GNOME;MATE;

I will investigate it tomorrow.

I tested:

`/sbin/yast2 sw_single %F` works if it is started as root. But problem is `xdg-su -c "..."`, because it disrupts arguments while transferring to /sbin/yast2

so maybe using ' instead of " should fix it? -- To unsubscribe, e-mail: yast-devel+unsubscribe@opensuse.org To contact the owner, e-mail: yast-devel+owner@opensuse.org

2016.02.05 13:55, opensuse.lietuviu.kalba rašė:

Strange, that `xsu` (/usr/bin/xsu) exist (at least inmy openSUSE 13.2 system), but `cnf xsu` returns no rezult about package of it. I doubt: /usr/bin/xsu is suposed to exist in every openSUSE system?

I found `xsu` in gksu package, but this does not exist in default repositories, It came for me from obs:GNOME repo (but I don't remember when and why I installed it)... -- To unsubscribe, e-mail: yast-devel+unsubscribe@opensuse.org To contact the owner, e-mail: yast-devel+owner@opensuse.org

Dear, YaST developers, If you agree, please consider to realize these suggestions, because I don't know how to to this properly. And mark https://features.opensuse.org/319376 as done. 2016.02.06 15:06, opensuse.lietuviu.kalba rašė:

I suggest add script https://build.opensuse.org/package/view_file/home:embar-:Lietukas/yast2-rpm-...

into https://github.com/yast/yast-packager as /usr/bin/yast2-rpm-handler

and add MimeType=application/x-rpm;application/x-redhat-package-manager; Exec=yast2-rpm-handler %F in https://github.com/yast/yast-packager/blob/master/desktop/yast2-packager.des...

-- To unsubscribe, e-mail: yast-devel+unsubscribe@opensuse.org To contact the owner, e-mail: yast-devel+owner@opensuse.org