forked from sebastiandev/zipper
-
Notifications
You must be signed in to change notification settings - Fork 1
pull upstream changes #4
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
Open
NevesLucas
wants to merge
90
commits into
NevesLucas:master
Choose a base branch
from
sebastiandev:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
zlib package in ubuntu is zlib1g
Update README.md
Add scripts for continuous integration: * Travis: gcc and clang for Linux 17.04 and OS X. * appVeyor: MinGW: not yet ok (cmake fails locating zlib). Note: tested for 64-bits architectures.
c59b0ef to
0124c6e
Compare
Fix Ubuntu version in Travis-CI. Exclude compiling with g++ on OSX because they only use clang. Fix Travis-CI badge linked on dev branch instead of master.
* added config, target export to CMake * UNIX, Windows have same Zipper library name
…ot rewind the file correctly. This is now fixed in tools.cpp so the password can be passed onto the Impl::add(..) method.
…ot rewind the file correctly. This is now fixed in tools.cpp so the password can be passed onto the Impl::add(..) method. (#47)
…debug postfix (#49)
Allow to add timestamp for archive streams.
* Add new flag to save file prefix This flag allows you to not lose file prefix (e.g. ./some/dir/before/file.txt) when we use add zipper::Zipper::add()
…c lib - Install a pkg-config file to avoid to know the path of includes and libraries for non CMake users. - Force by default to compile the shared library. - Install a second static library with the name libZipper.a (the 1st is still name libZipper-static.a) but this is not Unix way to name a static lib (ie I dunno make pkg-config knows it). - Read the VZERSION.txt file
Zipper: prevent from adding / \ * chars Unzipper: check if extracted file will be outside the target folder and forbid this case. Unzipper: does not allow smashing existing files. Add canonical path utility function
…n for unknow archi
Will be made for Zipper version 2.X This reverts commit 95078ca.
Will be done for zipper-v2 since this breaks the behavior. I'll let version 1.x as it and will rework for the version 2.x This reverts commit 9956824.
a5a216a to
10bb289
Compare
ioapi.h needs to be included before iowin32.h as iowin32.h (a minizip header pulled in as a submodule) references definitions in ioapi.h that are not defined in iowin32.h. Ideally this would be fixed in minizip's iowin32.h but simply including before #include-ing also fixes the issue at the zipper project level.
e5eae28 to
707dc39
Compare
60c9265 to
50ec4dd
Compare
2fa9fe8 to
87b14a4
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.