Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

makejdk-any-platform.sh -s, -d parameters do not work as they are supposed to #2058

Open
sxa opened this issue Sep 7, 2020 · 15 comments
Open
Assignees
Labels
bug Issues that are problems in the code as reported by the community good first issue Issues that have been deemed "easy" and excellent for new contributors

Comments

@sxa
Copy link
Member

sxa commented Sep 7, 2020

Using -s which is expected to be given as an absolute PATH does not work - it gets appended to the workspace directory where makejdk-any-platform.sh is invoked. This example output was from trying to set them both to directories under /home/sxa/jdk11:

Didn't find any existing openjdk repository at /home/sxa/openjdk-build/workspace/home/sxa/jdk11/src//home/sxa/jdk11/src so cloning the source to openjdk
git clone --depth=1 https://github.com/adoptopenjdk/openjdk-jdk11u.git /home/sxa/openjdk-build/workspace//home/sxa/jdk11/src/src
Cloning into '/home/sxa/openjdk-build/workspace//home/sxa/jdk11/src/src'...
checkoutRequiredCodeToBuild:
  workspace = /home/sxa/openjdk-build/workspace//home/sxa/jdk11/src/src
  BUILD_VARIANT = hotspot
  TAG = 
  BRANCH = dev

Same is true of -d rendering both more or less useless.

@sxa sxa added the bug Issues that are problems in the code as reported by the community label Sep 7, 2020
@sxa
Copy link
Member Author

sxa commented Sep 7, 2020

Also -T cannot be a full path to the location to put the resulting tarball - if you try you get a message like this:

Moving the artifact to /home/sxa/openjdk-build/workspace//home/sxa/jdk11/build
mv: cannot move `/home/sxa/openjdk-build/workspace/home/sxa/jdk11/src/src/build/linux-x86_64-normal-server-release/images/OpenJDK.tar.gz' to `/home/sxa/openjdk-build/workspace//home/sxa/jdk11/build//home/sxa/MyOpenJDK11.tar.gz': No such file or directory

@M-Davies M-Davies added the good first issue Issues that have been deemed "easy" and excellent for new contributors label Sep 8, 2020
@M-Davies M-Davies added good first issue - candidate This label defines that the issues can become a "good first issue" - must be reviewed and removed good first issue Issues that have been deemed "easy" and excellent for new contributors labels Sep 24, 2020
@hendrikebbers
Copy link
Member

@sxa Can you add some information how the error can be reproduces? If the error can be easily reproduced by a newcomer we can create a "good first issue" out of this one :)

@andrew-m-leonard andrew-m-leonard added the good first issue Issues that have been deemed "easy" and excellent for new contributors label Oct 10, 2022
@andrew-m-leonard
Copy link
Contributor

BUILD_ARGS of
--destintion
--source
don't act as expected.
We need to check the behavior of using the makejdk-any-platform.sh with these options:
see:

"--destination" | "-d" )

@andrew-m-leonard
Copy link
Contributor

Needs investigation as to whether these arguments work?

@zdtsw zdtsw removed the good first issue - candidate This label defines that the issues can become a "good first issue" - must be reviewed label Oct 10, 2022
@julian55455
Copy link
Contributor

Hi @smlambert can I take on this?

@julian55455
Copy link
Contributor

julian55455 commented Oct 12, 2022

Needs investigation as to whether these arguments work?

@andrew-m-leonard @zdtsw which command do we have to run with these parameters?

@zdtsw
Copy link
Contributor

zdtsw commented Oct 12, 2022

Needs investigation as to whether these arguments work?

@andrew-m-leonard @zdtsw which command do we have to run with these parameters?

makejdk-any-platform.sh -s and -d are the parameters for it.

@julian55455
Copy link
Contributor

makejdk-any-platform.sh -s and -d are the parameters for it.

@zdtsw Wondering 😮 why it isn't picking when I run it from the root directory of the project

@zdtsw
Copy link
Contributor

zdtsw commented Oct 12, 2022

makejdk-any-platform.sh -s and -d are the parameters for it.

@zdtsw Wondering 😮 why it isn't picking when I run it from the root directory of the project

Care to post your log? how you run it?

@julian55455
Copy link
Contributor

Care to post your log? how you run it?

git

@zdtsw there it is

@zdtsw
Copy link
Contributor

zdtsw commented Oct 12, 2022

Care to post your log? how you run it?

git

@zdtsw there it is

permisson on the file? change to executable and see how it goes without ./

@julian55455
Copy link
Contributor

julian55455 commented Oct 12, 2022

permisson on the file? change to executable and see how it goes without

@zdtsw Actually I had checked earlier and it has all the file rights read, write and execute

Btw am running Ubuntu 20.04
git2

@sxa @zdtsw I actaully tried to spin from another machine today and it didn't help. Could this be another issue?

@zdtsw
Copy link
Contributor

zdtsw commented Oct 13, 2022

permisson on the file? change to executable and see how it goes without

@zdtsw Actually I had checked earlier and it has all the file rights read, write and execute

Btw am running Ubuntu 20.04 git2

@sxa @zdtsw I actaully tried to spin from another machine today and it didn't help. Could this be another issue?

I do not think this is an issue. At least to call a script, I always use the path or add sh (rarely)
The only case as far as I know is, the executable has been added into the PATH, or it is a linux built-in command, which can skip ./

@julian55455
Copy link
Contributor

julian55455 commented Oct 13, 2022

@sxa can shade more light on how to reproduce this bug possibly

@zdtsw zdtsw moved this to In Progress in Outreachy December 2022 Oct 13, 2022
@sxa
Copy link
Member Author

sxa commented Oct 18, 2022

makejdk-any-platform.sh requires a parameter to specify which JDK to build. If you run it without any parameters it will say:

[sxa@fedora openjdk-build]$ ./makejdk-any-platform.sh
Starting ./makejdk-any-platform.sh to configure, build (Adoptium)OpenJDK binary
Please provide a java version to build as an argument
[sxa@fedora openjdk-build]$ 

So the normal thing to do would be to run ./makejdk-any-platform.sh jdk.

To use -s (which is to specify a directory on your local machine to clone to) you need to tell it which directory to use for the -s option e.g.

[sxa@fedora openjdk-build]$ ./makejdk-any-platform.sh
Starting ./makejdk-any-platform.sh to configure, build (Adoptium)OpenJDK binary
Please provide a java version to build as an argument
[sxa@fedora openjdk-build]$ 

If you run ./makejdk-any-platform.sh -h it will give you more information on the parameters, or just look at the source of that help page which is in https://github.com/adoptium/temurin-build/blob/master/makejdk-any-platform.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issues that are problems in the code as reported by the community good first issue Issues that have been deemed "easy" and excellent for new contributors
Projects
None yet
Development

No branches or pull requests

6 participants