All of lore.kernel.org
 help / color / mirror / Atom feed
From: Yann E. MORIN <yann.morin.1998@free.fr>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 1/4] openjdk: new package
Date: Sun, 25 Feb 2018 20:39:57 +0100	[thread overview]
Message-ID: <20180225193957.GE2276@scaer> (raw)
In-Reply-To: <20180207095110.GJ20598@australia.be.alcatel-lucent.com>

Thomas, All,

On 2018-02-07 10:51 +0100, Thomas De Schampheleire spake thusly:
> On Tue, Feb 06, 2018 at 11:15:14PM +0100, Arnout Vandecappelle wrote:
> >  You can achieve this locally by adding...
> > [snip]
> > >> +OPENJDK_MAKE_ENV += X11_NOT_NEEDED=yes CUPS_NOT_NEEDED=yes ALSA_NOT_NEEDED=yes \
> > >> +		    PULSE_NOT_NEEDED=yes \
> > 			    PLATFORM= JAVA_HOME= OS= \
> > >> +		    BUILD_LD="$(HOSTCC)"
> 
> For JAVA_HOME this indeed makes perfect sense.
> 
> However, OS and PLATFORM are more generic variables that may conflict with
> various packages. I have seen issues with 'OS' for other (proprietary 3rd party)
> packages as well, I think likewise for PLATFORM.
> 
> So, perhaps the change for OS and PLATFORM could become a separate patch for
> inclusion in Buildroot? Would you agree?

Well, you may try, but if it is about third-party, proprietary, private
packages, then my position is that those packages should ensure their
environent is clean, doing as Arnout suggested...

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

  reply	other threads:[~2018-02-25 19:39 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-02-06 14:39 [Buildroot] [PATCH 0/4] openjdk Thomas De Schampheleire
2018-02-06 14:39 ` [Buildroot] [PATCH 1/4] openjdk: new package Thomas De Schampheleire
2018-02-06 14:52   ` Thomas De Schampheleire
2018-02-06 22:15     ` Arnout Vandecappelle
2018-02-07  9:51       ` Thomas De Schampheleire
2018-02-25 19:39         ` Yann E. MORIN [this message]
2018-02-06 14:39 ` [Buildroot] [PATCH 2/4] openjdk size optimization: disable debug info Thomas De Schampheleire
2018-02-06 14:39 ` [Buildroot] [PATCH 3/4] openjdk size optimization: link dynamically with system libstdc++ Thomas De Schampheleire
2018-02-06 14:39 ` [Buildroot] [PATCH 4/4] openjdk size optimization: compress JAR files Thomas De Schampheleire
2018-04-20 11:37 ` [Buildroot] [PATCH 0/4] openjdk Thomas Petazzoni
2018-04-20 12:33   ` Yann E. MORIN
2018-04-20 12:52     ` Matthew Weber

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=20180225193957.GE2276@scaer \
    --to=yann.morin.1998@free.fr \
    --cc=buildroot@busybox.net \
    /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 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.