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.

Sign up for GitHub

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

Jump to bottom

Add Zip static PIE build #70

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add Zip static PIE build #70

wants to merge 1 commit into from

Conversation

anindyasen
Copy link

  1. Added zip application as a static-pie build
  2. A build.sh script is created to download, extract, patch and compile the application.
  3. rootfs, filesystem to store file in unikernel.
  4. Patch has the changes required in zip application to build it as static-pie.

StefanJum
StefanJum requested changes Oct 15, 2022
View reviewed changes
Copy link
Member

@StefanJum StefanJum left a comment

Choose a reason for hiding this comment

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

Looks good, thanks,
Please see the comments and also sign off your commit.

zip/README.md Outdated Show resolved Hide resolved
zip/README.md Outdated Show resolved Hide resolved
zip/README.md Outdated Show resolved Hide resolved
zip/build.sh Outdated Show resolved Hide resolved
StefanJum
StefanJum requested changes Oct 20, 2022
View reviewed changes
Copy link
Member

@StefanJum StefanJum left a comment

Choose a reason for hiding this comment

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

Looks good, just a few more comments.

zip/README.md Outdated Show resolved Hide resolved
zip/README.md Outdated Show resolved Hide resolved
StefanJum
StefanJum requested changes Oct 21, 2022
View reviewed changes
Copy link
Member

@StefanJum StefanJum left a comment

Choose a reason for hiding this comment

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

Thank you @anindyasen, hopefully these are the last comments, fix them and I will approve the pr 🚀

zip/README.md Outdated Show resolved Hide resolved
zip/README.md Outdated Show resolved Hide resolved
@StefanJum
Copy link
Member

Looks good but please remove the Co-authored-by: Jumarea Stefan <69107167+StefanJum@users.noreply.github.com> line from the commit message.

@anindyasen
Add Zip static PIE build
03709ab 
Signed-off-by: anindya_sen <anindyasen007@gmail.com>
StefanJum
StefanJum approved these changes Oct 21, 2022
View reviewed changes
Copy link
Member

@StefanJum StefanJum left a comment

Choose a reason for hiding this comment

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

All good now, thank you.

Reviewed-by: Stefan Jumarea stefanjumarea02@gmail.com

StefanJum
StefanJum approved these changes Oct 21, 2022
View reviewed changes
Copy link
Member

@StefanJum StefanJum left a comment

Choose a reason for hiding this comment

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

All good now, thank you.

Reviewed-by: Stefan Jumarea stefanjumarea02@gmail.com

@anindyasen anindyasen mentioned this pull request Oct 21, 2022
Open
@razvand razvand added the kind/hackathon Hackathon challenge label Oct 23, 2022
@razvand razvand linked an issue Oct 23, 2022 that may be closed by this pull request
Add Zip static PIE build #30
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@StefanJum StefanJum StefanJum approved these changes

Assignees

@razvand razvand

Labels
kind/hackathon Hackathon challenge
Projects
(Binary) Application Compatibility
Status: No status
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add Zip static PIE build
3 participants
@anindyasen @StefanJum @razvand