From: aduskett at gmail.com <aduskett@gmail.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v2 2/3] package/openjdk: copy all directories and files when installing
Date: Sat, 18 Apr 2020 12:07:00 -0700 [thread overview]
Message-ID: <20200418190701.2629613-2-aduskett@gmail.com> (raw)
In-Reply-To: <20200418190701.2629613-1-aduskett@gmail.com>
From: Adam Duskett <Aduskett@gmail.com>
Several directories and files are currently not installed during the
target installation, these include:
- conf
Several configuration files, including security configuration files which
may be necessary for running various java applications.
- legal
This directory contains legal notices that some java applications may
require, as they may print legal information and will throw exceptions at
runtime if the legal files are not present on the system.
- release
This file contains a list of modules included in the image.
Because these directories take up less than of megabyte extra, it is not an
issue to install all of them.
Signed-off-by: Adam Duskett <Aduskett@gmail.com>
---
Changes v1 -> v2:
- Added a better explination for why the legal directory is necessary.
package/openjdk/openjdk.mk | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/package/openjdk/openjdk.mk b/package/openjdk/openjdk.mk
index a06a8a7307..ffa62dd10d 100644
--- a/package/openjdk/openjdk.mk
+++ b/package/openjdk/openjdk.mk
@@ -123,10 +123,8 @@ endef
# which makes manual installation necessary.
define OPENJDK_INSTALL_TARGET_CMDS
mkdir -p $(TARGET_DIR)$(OPENJDK_INSTALL_BASE)
- cp -dpfr $(@D)/build/linux-*-release/images/jre/bin/ \
- $(TARGET_DIR)$(OPENJDK_INSTALL_BASE)
- cp -dpfr $(@D)/build/linux-*-release/images/jre/lib/ \
- $(TARGET_DIR)/$(OPENJDK_INSTALL_BASE)/
+ cp -dpfr $(@D)/build/linux-*-release/images/jre/* \
+ $(TARGET_DIR)$(OPENJDK_INSTALL_BASE)/
cd $(TARGET_DIR)/usr/bin && ln -snf ../..$(OPENJDK_INSTALL_BASE)/bin/* .
endef
--
2.25.2
next prev parent reply other threads:[~2020-04-18 19:07 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-04-18 19:06 [Buildroot] [PATCH v2 1/3] package/openjdk: fix installation with merged usr directories aduskett at gmail.com
2020-04-18 19:07 ` aduskett at gmail.com [this message]
2020-04-21 17:12 ` [Buildroot] [PATCH v2 2/3] package/openjdk: copy all directories and files when installing Ryan Barnett
2020-05-08 7:25 ` Peter Korsgaard
2020-04-18 19:07 ` [Buildroot] [PATCH v2 3/3] packageJopenjdk: add support for building the full jdk aduskett at gmail.com
2020-04-21 17:13 ` Ryan Barnett
2020-04-22 19:33 ` Yann E. MORIN
2020-04-21 17:12 ` [Buildroot] [PATCH v2 1/3] package/openjdk: fix installation with merged usr directories Ryan Barnett
2020-04-21 21:20 ` Yann E. MORIN
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=20200418190701.2629613-2-aduskett@gmail.com \
--to=aduskett@gmail.com \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox