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

Fix Build with Dockerfile #161

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

alibabakhanlu12
Copy link

i wanted to use composer in nexus and i find this repo. i built this app using docker but i run into the problem and build failed . i searched the error i had and i find that with adding and installing git in dockerfile my problem resolved

This pull request makes the following changes:

  • adding git in Dockerfile

i wanted to use composer in nexus and i find this repo.
i built this app using docker but i run into the problem and build failed . i searched the error i had  and i find that with adding and installing git in dockerfile my problem resolved
Dockerfile Show resolved Hide resolved
@paul-botsco-2-0 paul-botsco-2-0 bot added the 🧐 cla not signed The CLA needs to be signed label Oct 1, 2024
@paul-botsco-2-0
Copy link

Thanks for the contribution. Before we can merge this, we need @alibabakhanlu12, @alibabakhanlu12 to sign the Contributor License Agreement

@paul-botsco-2-0 paul-botsco-2-0 bot added 😍 cla signed The CLA is signed and removed 🧐 cla not signed The CLA needs to be signed labels Oct 1, 2024
@CybotTM
Copy link

CybotTM commented Nov 5, 2024

Thanks for this plugin at all and this PR.

Would it be possible to consider renaming this PR to something like 'Fix Build with Docker' to make its purpose even clearer? It helps other to find it while it is not being merged.

Adding the whole error message to make it easier to find when searching for it:

tl;dr: git ... No such file or directory

[INFO] Executing: /bin/sh -c cd '/nexus-repository-composer' && 'git' 'rev-parse' '--verify' 'HEAD'
[INFO] Working directory: /nexus-repository-composer
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary for org.sonatype.nexus.plugins:composer-parent-parent 0.1.6-SNAPSHOT:
[INFO]
[INFO] org.sonatype.nexus.plugins:composer-parent-parent .. FAILURE [ 17.349 s]
[INFO] nexus-repository-composer .......................... SKIPPED
[INFO] nexus-repository-composer-it ....................... SKIPPED
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  01:06 min
[INFO] Finished at: 2024-11-05T07:08:08Z
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.codehaus.mojo:buildnumber-maven-plugin:1.4:create (build-identity) on project composer-parent: Cannot get the revision information from the scm repository :
[ERROR] Exception while executing SCM command.: Error while executing command. Error while executing process. Cannot run program "git" (in directory "/nexus-repository-composer"): error=2, No such file or directory

Additionally, we would be grateful if this PR could be prioritized for merging, as the current Docker build process as documented in the README is unfortunately broken and requires these changes to function correctly again.

Thank you very much for your attention to this!

@alibabakhanlu12 alibabakhanlu12 changed the title Update Dockerfile Fix Build with Docker Nov 8, 2024
@alibabakhanlu12 alibabakhanlu12 changed the title Fix Build with Docker Fix Build with Dockerfile Nov 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
😍 cla signed The CLA is signed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants