unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: John Darrington <john@darrington.wattle.id.au>
To: Ludovic Court??s <ludo@gnu.org>
Cc: guix-devel@gnu.org
Subject: Re: [PATCH 1/4] gnu: Add XFILESEARCH path to profiles' environment.
Date: Tue, 29 Nov 2016 19:16:00 +0100	[thread overview]
Message-ID: <20161129181559.GA30970@jocasta.intra> (raw)
In-Reply-To: <878ts273to.fsf@gnu.org>

[-- Attachment #1: Type: text/plain, Size: 2577 bytes --]

On Tue, Nov 29, 2016 at 03:34:11PM +0100, Ludovic Court??s wrote:
     John Darrington <john@darrington.wattle.id.au> skribis:

     > We can cut down on the size of this string iff we can somehow
     > guarantee that no package ever ships a file in any of those locations.
     >
     > Some "solutions" (in my order of preference) are:
     >
     > * The size of the above list can be halved, by dropping either the
     > .../lib/... or the .../share/... items - we just have to then make
     > sure that no package ships resource files in the one we drop.  Historically,
     > resource files were always in .../lib (as still are all official
     > sources from x.org) but recently third party packages have started
     > putting them in .../share.
     >
     > * I *think* we could also get away with further reducing the set to 
     >     "$HOME/.guix-profile/lib/X11/%T/%N%S:
     >     /run/current-system/profile/lib/X11/%T/%N%S"
     > because, all the Xt dependent packages I've seen so far, put their
     > resource files there.  However, we cannot know what might get added
     > in the future.
     
     Right, if we do both, that???s already much better.
     
     > * Hack the hard coded defaults in the libXt source to use the profile
     > settings instead of /usr
     
     Maybe we should just do that, no?  It???d be a local change, it would
     achieve the same effect, and it would provide a good default.
     
     WDYT?

It sounds unreliable to me.  What would we set it to?
$GUIX_PROFILE/...  $HOME/.guix-profile/... /run/current-system/profile/...
There's no guarantee that the current profile is any of those.

Something similar to the first solution seems prefereable to me.  Maybe:

 Set XFILESEARCHPATH to "$HOME/.guix-profile/lib/X11/%T/%N%S:/run/current-system/profile/lib/X11/%T/%N%S"
  (we can substitute "share" for "lib" if we want to keep the FHS people happy).

 Then we audit all users of libxt to make sure they put their resource files in lib/X11/app-defaults.
 Most packages have a --with-app-defaults configure option (which currently we are in many cases
 either not using or misusing).

 We could also consider adding a build stage to make sure that packages which depend on libXt put
 their resource files in the "right" place.

     

-- 
Avoid eavesdropping.  Send strong encrypted email.
PGP Public key ID: 1024D/2DE827B3 
fingerprint = 8797 A26D 0854 2EAB 0285  A290 8A67 719C 2DE8 27B3
See http://sks-keyservers.net or any PGP keyserver for public key.


[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 181 bytes --]

  reply	other threads:[~2016-11-29 18:16 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-11-25 19:08 [PATCH 1/4] gnu: Add XFILESEARCH path to profiles' environment John Darrington
2016-11-25 19:08 ` [PATCH 2/4] gnu: Add editres John Darrington
2016-11-27 21:22   ` Ludovic Courtès
2016-11-25 19:08 ` [PATCH 3/4] gnu: Remove wrappers from xfd and xfontsel John Darrington
2016-11-25 19:08 ` [PATCH 4/4] gnu: Xfig: Set XAPPLOADDIR John Darrington
2016-11-27 21:22 ` [PATCH 1/4] gnu: Add XFILESEARCH path to profiles' environment Ludovic Courtès
2016-11-28 13:19   ` John Darrington
2016-11-29 14:34     ` Ludovic Courtès
2016-11-29 18:16       ` John Darrington [this message]
2016-11-29 21:42         ` Ludovic Courtès
2016-11-30  5:36           ` John Darrington
2016-12-02 16:25           ` [PATCH] gnu: Patch libxt's default search path John Darrington
2016-12-04 21:19             ` Ludovic Courtès

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://guix.gnu.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20161129181559.GA30970@jocasta.intra \
    --to=john@darrington.wattle.id.au \
    --cc=guix-devel@gnu.org \
    --cc=ludo@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).