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

[DNM] test manpages from docker/cli gha-validate pr #620

Closed
wants to merge 2 commits into from

Conversation

crazy-max
Copy link
Member

@crazy-max crazy-max marked this pull request as draft January 27, 2022 14:30
@crazy-max crazy-max force-pushed the test-cli-validate-pr branch 3 times, most recently from 519fa99 to 1e9c6e8 Compare January 27, 2022 17:58
@crazy-max
Copy link
Member Author

On CentOS Jenkins nodes Git is quite old:

#12 68.22   git.aarch64 0:1.8.3.1-23.el7_8

https://ci-next.docker.com/public/blue/organizations/jenkins/docker-ce-packaging/detail/PR-620/4/pipeline/865/#step-1037-log-372

+ go mod vendor -modfile=vendor.mod
go: github.com/theupdateframework/notary@v0.7.1-0.20210315103452-bf96a202a09a requires
	github.com/docker/go@v1.5.1-1.0.20160303222718-d30aec9fd63c: invalid pseudo-version: git fetch --unshallow -f origin in /go/pkg/mod/cache/vcs/48fbd2dfabec81f4c93170677bfc89087d4bec07a2d08f6ca5ce3d17962677ee: exit status 128:
	fatal: git fetch-pack: expected shallow list

@thaJeztah Not sure where to update the Dockerfile

@crazy-max crazy-max force-pushed the test-cli-validate-pr branch 4 times, most recently from 7d77711 to 6208a41 Compare January 27, 2022 19:36
Comment on lines +21 to +39
# Use recent Git version
ARG GIT_VERSION="2.30.0"
RUN yum install -y \
curl-devel \
expat-devel \
gettext-devel \
openssl-devel \
perl-CPAN \
perl-devel \
wget \
zlib-devel \
&& yum --setopt=group_package_types=mandatory,default,optional groupinstall -y "Development Tools" \
&& wget https://github.com/git/git/archive/v${GIT_VERSION}.tar.gz -O git.tar.gz \
&& tar -zxf git.tar.gz \
&& cd git-* \
&& make prefix=/usr/local all \
&& make prefix=/usr/local install \
&& rm -rf /git* \
&& git --version
Copy link
Member Author

Choose a reason for hiding this comment

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

Ok needs to update Git for CentOS 7. I prefer to build from source as some third party repositories are not reliable and trusted. It takes a few minutes to build Git, so I think it's fine.

@crazy-max
Copy link
Member Author

PTAL @thaJeztah about #620 (comment) if you have a better alternative.

Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
@crazy-max crazy-max closed this Mar 4, 2022
@crazy-max crazy-max deleted the test-cli-validate-pr branch March 4, 2022 17:54
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.

1 participant