From: Ricardo Wurmus <rekado@elephly.net>
To: Chris Marusich <cmmarusich@gmail.com>
Cc: Guix-devel <guix-devel@gnu.org>,
Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de>
Subject: Re: broken references in jar manifests
Date: Mon, 19 Mar 2018 05:47:11 +0100 [thread overview]
Message-ID: <877eq8u0xc.fsf@elephly.net> (raw)
In-Reply-To: <87k1u8u1zg.fsf@gmail.com>
Chris Marusich <cmmarusich@gmail.com> writes:
>>> >> we have a problem with jar manifests. When we use the Class-Path
>>> >> property to ensure that an executable can find its dependencies on the
>>> >> class path at run-time, we end up with a manifest like this:
>>> >>
>>> >> --8<---------------cut here---------------start------------->8---
>>> >> Manifest-Version: 1.0
>>> >> Class-Path: /gnu/store/i28vi94r8z9f0x02zgkrv87w16ibmqkw-java-htsjdk-2.
>>> >> 10.1/share/java/htsjdk.jar
>>> >> Created-By: 1.8.0_151 (Oracle Corporation)
>>> >> Main-Class: picard.cmdline.PicardCommandLine
>>> >> --8<---------------cut here---------------end--------------->8---
>>> >>
>>> >> Note that the Class-Path property is broken into two lines. This means
>>> >> that the reference scanner will miss it and grafting will fail.
>
> That is quite unfortunate. :-(
>
>>> An alternative to recording full references in the manifest file is to
>>> install a “lib” directory that contains symlinks to dependencies. The
>>> manifest can then contain relative paths to these symlinks.
>>>
>>>
>> I guess I would prefer to do this instead.
>
> Sounds good to me, too.
Actually, I think this wouldn’t work after all. When the references in
the Class-Path property are split up they would be overlooked when
grafting. The references in the separate custom file would be grafted.
The result is that the applications would keep trying to load the old,
ungrafted jars while Guix would think that the package only retained
references to the grafted packages.
Equally bad: the garbage collector would trash these ungrafted
references as it cannot detect them, thus breaking the application.
--
Ricardo
GPG: BCA6 89B6 3655 3801 C3C6 2150 197A 5888 235F ACAC
https://elephly.net
next prev parent reply other threads:[~2018-03-19 4:48 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-03-01 17:11 broken references in jar manifests Ricardo Wurmus
2018-03-01 18:32 ` Gábor Boskovits
2018-03-01 18:54 ` Ricardo Wurmus
2018-03-01 19:08 ` Gábor Boskovits
2018-03-01 19:52 ` Ricardo Wurmus
2018-03-19 4:24 ` Chris Marusich
2018-03-19 4:47 ` Ricardo Wurmus [this message]
2018-03-02 13:18 ` Ludovic Courtès
2018-03-20 10:28 ` Danny Milosavljevic
2018-03-20 10:50 ` julien lepiller
2018-03-20 12:14 ` Danny Milosavljevic
2018-03-20 13:47 ` Ricardo Wurmus
2018-03-21 21:07 ` Danny Milosavljevic
2018-03-21 22:58 ` Ricardo Wurmus
2018-03-25 10:19 ` [PATCH] Create INDEX.LIST; Was: Fix " Danny Milosavljevic
2018-04-01 22:12 ` Chris Marusich
2018-04-18 13:25 ` Danny Milosavljevic
2018-05-06 18:09 ` Danny Milosavljevic
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
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=877eq8u0xc.fsf@elephly.net \
--to=rekado@elephly.net \
--cc=cmmarusich@gmail.com \
--cc=guix-devel@gnu.org \
--cc=ricardo.wurmus@mdc-berlin.de \
/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 external index
https://git.savannah.gnu.org/cgit/guix.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.