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

build: add -buildmode=pie to builds #412

Merged
merged 1 commit into from
Mar 24, 2022
Merged

Conversation

rchincha
Copy link
Contributor

Signed-off-by: Ramkumar Chinchani rchincha@cisco.com

What type of PR is this?

build

Which issue does this PR fix:

What does this PR do / Why do we need it:

Adds -buildmode=pie to build PIE binaries.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@rchincha rchincha added this to In progress in zot-core via automation Feb 16, 2022
@rchincha rchincha added this to the v1.3.8 milestone Feb 16, 2022
@codecov
Copy link

codecov bot commented Feb 16, 2022

Codecov Report

Merging #412 (8fd9086) into main (b2a4388) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #412   +/-   ##
=======================================
  Coverage   84.41%   84.41%           
=======================================
  Files          51       51           
  Lines       10548    10548           
=======================================
  Hits         8904     8904           
  Misses       1287     1287           
  Partials      357      357           

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@rchincha rchincha force-pushed the fix-build branch 2 times, most recently from ab384c8 to aa2afd5 Compare February 16, 2022 22:51
@rchincha rchincha removed this from the v1.3.8 milestone Feb 17, 2022
zot-core automation moved this from In progress to Reviewer approved Feb 23, 2022
peusebiu
peusebiu previously approved these changes Feb 23, 2022
zot-core automation moved this from Reviewer approved to Review pending Mar 22, 2022
@rchincha rchincha force-pushed the fix-build branch 2 times, most recently from a187e94 to df7cf6f Compare March 24, 2022 19:03
@rchincha rchincha added this to the v1.4.0 milestone Mar 24, 2022
This makes zot a dynamic binary, so the container base image has to be
changed accordingly

Signed-off-by: Ramkumar Chinchani <rchincha@cisco.com>
zot-core automation moved this from Review pending to Reviewer approved Mar 24, 2022
@rchincha rchincha merged commit 554e99a into project-zot:main Mar 24, 2022
zot-core automation moved this from Reviewer approved to Done Mar 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
zot-core
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants