On Thu, 4 Feb 2016 15:36:59 +0100 Josef Reidinger <[email protected]> wrote:
> On Thu, 4 Feb 2016 16:22:18 +0200 > "opensuse.lietuviu.kalba" <[email protected]> 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" <[email protected]> > > >> wrote: > > >>> 2016.02.04 15:14, Josef Reidinger rašė: > > >>>> On Thu, 4 Feb 2016 15:06:12 +0200 > > >>>> "opensuse.lietuviu.kalba" <[email protected]> > > >>>> 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-handler > > >>>>> > > >>>>> > > >>>>> [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 > > >>> 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-handler/yast2-rpm-handler.desktop?expand=1 > > >> > > >> > > >> 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.desktop > > >> > > >> > > >> > > >> 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.desktop > > > > > > > > > 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-latest.html#exec-variables 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-latest.html#example As Martin Vidner, author of script is on this mailing list I will let him to also comment it. Josef -- To unsubscribe, e-mail: [email protected] To contact the owner, e-mail: [email protected]
