Downgrades in xfontsel and other packages using app-defaults

  • Open
  • quality assurance status badge
Details
2 participants
  • Alex Kost
  • Ludovic Courtès
Owner
unassigned
Submitted by
Alex Kost
Severity
normal

Debbugs page

A
A
Alex Kost wrote on 30 Aug 2017 12:08
(address . bug-guix@gnu.org)(name . John Darrington)(address . jmd@gnu.org)
87a82gx4a3.fsf@gmail.com
Hello, commit 5dc8dd07b¹ removed a wrapper from xfontsel package. And
now xfontsel looks "ugly" (not as intended). You can simply check it:

$ `guix build xfontsel`/bin/xfontsel

It doesn't look right, and also prints the following message:

Warning: app-defaults file not properly installed.

Now if you run it like this:

$ XAPPLRESDIR=/gnu/store/...-xfontsel-1.0.5/lib/X11/app-defaults /gnu/store/...-xfontsel-1.0.5/bin/xfontsel

it looks as it should.

Also the same downgrades probably take effect (I didn't check) with
'xfig' and 'xfd' packages (commits 97fedab1 and 5aae77e4).

So I think we should re-add the mentioned wrappers.


--
Alex
A
A
Alex Kost wrote on 31 Aug 2017 12:14
(name . John Darrington)(address . jmd@gnu.org)(address . 28298@debbugs.gnu.org)
87y3pzy2g4.fsf@gmail.com
John Darrington (2017-08-30 15:21 -0400) wrote:

Toggle quote (2 lines)
> It looks fine to me. I don't get that warning.

But did you actually tried what I wrote?:

$ `guix build xfontsel`/bin/xfontsel

Toggle quote (3 lines)
> The XAPPLRESDIR is no longer required because the libxt library has
> been patched.

Oh, I didn't know about this "libxt-guix-search-paths.patch". I think
now I understand: with this patch, xfontsel and other programs should
work only when they are installed from the usual guix profiles (system
and user ones), right?

In my opinion, this is the wrong approach. A user may install these
programs in non-standard profiles. For example, I don't use
"~/.guix-profile" at all; instead I use several other profiles, so there
is no chance for me to have 'xfontsel' work properly.

Also, I think a program should preferably work when it is started
directly from store.

So I strongly believe that using wrappers was the right approach, and we
should return to it.

--
Alex
L
L
Ludovic Courtès wrote on 2 Sep 2017 13:52
Re: bug#28298: Downgrades in xfontsel and other packages using app-defaults
(name . Alex Kost)(address . alezost@gmail.com)
87efrox1pj.fsf@gnu.org
Hi Alex & John!

Alex Kost <alezost@gmail.com> skribis:

Toggle quote (16 lines)
> John Darrington (2017-08-30 15:21 -0400) wrote:
>
>> It looks fine to me. I don't get that warning.
>
> But did you actually tried what I wrote?:
>
> $ `guix build xfontsel`/bin/xfontsel
>
>> The XAPPLRESDIR is no longer required because the libxt library has
>> been patched.
>
> Oh, I didn't know about this "libxt-guix-search-paths.patch". I think
> now I understand: with this patch, xfontsel and other programs should
> work only when they are installed from the usual guix profiles (system
> and user ones), right?

Or when $GUIX_PROFILE is set, AIUI.

Toggle quote (8 lines)
> In my opinion, this is the wrong approach. A user may install these
> programs in non-standard profiles. For example, I don't use
> "~/.guix-profile" at all; instead I use several other profiles, so there
> is no chance for me to have 'xfontsel' work properly.
>
> Also, I think a program should preferably work when it is started
> directly from store.

I agree, though that’s not always possible.

Toggle quote (3 lines)
> So I strongly believe that using wrappers was the right approach, and we
> should return to it.

As a preamble, I think it’s not just “right” or “wrong”, it’s different
solutions making different tradeoffs. :-)

IIRC I participated in the review of the libXt patch. I don’t really
remember the discussion, but I think it would be good rereading it
before making a decision, to make sure we don’t run in circles.

Thanks for looking into it!

Ludo’.
?
Your comment

Commenting via the web interface is currently disabled.

To comment on this conversation send an email to 28298@patchwise.org

To respond to this issue using the mumi CLI, first switch to it
mumi current 28298
Then, you may apply the latest patchset in this issue (with sign off)
mumi am -- -s
Or, compose a reply to this issue
mumi compose
Or, send patches to this issue
mumi send-email *.patch