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

chore: drop pinned versions for dockerfile #2660

Conversation

Chris53897
Copy link
Contributor

Will fix #2624
The other fixed versions are a problem today (they worked at the time at the creation of the issue). It looks that these versions are not available/reachable anymore on mac.

Are the fixed versions necessary?
If yes, i would suggest do duplicate the actual Docker-File Dockerfile-fallback-mac? as fallback for mac.

@@ -10,11 +10,11 @@ COPY --from=composer /usr/bin/composer /usr/bin/composer

RUN apt-get update \

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
RUN apt-get update \
# hadolint ignore=DL3008
RUN apt-get update \

suggestion: adding this line will allow unpinned package version : https:/hadolint/hadolint/wiki/DL3008

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@arnaud-deabreu Thanks for this info. Do you know, if there is a mode to use the pinned versions, with a fallback to unpinned versions?

If not, i guess it is the best, that i revert my changes. And only add your suggested commit (with a comment) to this PR.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you could try with something like this:

apt-get satisfy -qq --yes --no-install-recommends \
    "git (>= 1:2.30.2), git (<< 1:3)"

@Chris53897 Chris53897 changed the title chore: drop fixed versions for dockerfile chore: drop pinned versions for dockerfile Aug 19, 2023
@Chris53897
Copy link
Contributor Author

I will close this, as the linked PR solves this issue better.

@Chris53897 Chris53897 closed this Aug 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

docker-compose up does not run local on Mac M1
4 participants