public inbox for gentoo-java@lists.gentoo.org
 help / color / mirror / Atom feed
From: Joshua Nichols <nichoj@gentoo.org>
To: gentoo-java@lists.gentoo.org
Subject: Re: [gentoo-java] Running java-pkg_ensure-no-bundled-jars automatically with JAVA_PKG_STRICT
Date: Tue, 02 Jan 2007 20:53:07 -0500	[thread overview]
Message-ID: <459B0C83.3080207@gentoo.org> (raw)
In-Reply-To: <459A84B1.2000503@gentoo.org>

Petteri Räty wrote:
> Any objections to the attached patch? It will probably find some false
> positives but in those cases one can just use JAVA_PKG_STRICT= emerge.
>
> Regards,
> Petteri
>   
> ------------------------------------------------------------------------
>
> Index: java-pkg-2.eclass
> ===================================================================
> RCS file: /var/cvsroot/gentoo-x86/eclass/java-pkg-2.eclass,v
> retrieving revision 1.8
> diff -u -r1.8 java-pkg-2.eclass
> --- java-pkg-2.eclass	18 Dec 2006 10:18:56 -0000	1.8
> +++ java-pkg-2.eclass	2 Jan 2007 16:11:41 -0000
> @@ -84,6 +84,7 @@
>  }
>  
>  pre_src_compile() {
> +	is-java-strict && java-pkg_ensure-no-bundled-jars
>  	java-pkg-2_pkg_setup
>  }
>  
>   
I'm not sure if we should add this quite yet. Any idea how many packages 
this would affect?

Some problem areas I expect:

* Some packages are weird, and jar up their sources inside of their 
upstream distfile, which we then unpack. So, we'd either need to remove 
those jars after unpacking, or perhaps add some checks to ensure the 
bundled jars don't contain class files
* Some packages have jars for like tests, or examples, or whatnot. Maybe 
we need a variable to say what jars are ok?
* In some cases, we may be only replacing some jars with jar-from... but 
perhaps not all of them, or maybe they are meant to replace a jar, but 
the jar changed names, so both the symlink from jar-from and the 
original are around.

Whitelisting jars should take care of the first two points, and for the 
latter, well, I suppose those are the type of things this is intended to 
catch.

-- 
Joshua Nichols
Gentoo/Java Project Lead
-- 
gentoo-java@gentoo.org mailing list



  reply	other threads:[~2007-01-03  1:53 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-01-02 16:13 [gentoo-java] Running java-pkg_ensure-no-bundled-jars automatically with JAVA_PKG_STRICT Petteri Räty
2007-01-03  1:53 ` Joshua Nichols [this message]
2007-01-06 13:09   ` Petteri Räty
2007-01-09 16:13     ` Petteri Räty

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=459B0C83.3080207@gentoo.org \
    --to=nichoj@gentoo.org \
    --cc=gentoo-java@lists.gentoo.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.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox