Skip to content

Bump minizip submodule#20

Merged
nextgeniuspro merged 4 commits into
nextgeniuspro:masterfrom
psyinf:bump_minizip_submodule
Feb 4, 2025
Merged

Bump minizip submodule#20
nextgeniuspro merged 4 commits into
nextgeniuspro:masterfrom
psyinf:bump_minizip_submodule

Conversation

@psyinf

@psyinf psyinf commented Jan 27, 2025

Copy link
Copy Markdown
Contributor

Moving towards fixed minizip

Also: would you mind creating a new release/tag

psyinf and others added 4 commits January 27, 2025 22:38
* Add CloseFile override for ZipFileSystem

add gitignore for out (default for CPM/FetchContent/Visual Studio in-source builds)

* Optional build of examples

added library alias target for conformity
Modified examples for in-lib build

* using binary mode for opening NativeFile

* Update miniz-cpp submodule url

* Update readme

---------

Co-authored-by: Yev <evg1985@gmail.com>
* Add CloseFile override for ZipFileSystem

add gitignore for out (default for CPM/FetchContent/Visual Studio in-source builds)

* Optional build of examples

added library alias target for conformity
Modified examples for in-lib build

* using binary mode for opening NativeFile

* Update miniz-cpp submodule url

* Update readme

---------

Co-authored-by: Yev <evg1985@gmail.com>
@psyinf
psyinf marked this pull request as ready for review January 27, 2025 21:58
@nextgeniuspro
nextgeniuspro merged commit 7bcffe3 into nextgeniuspro:master Feb 4, 2025
@nextgeniuspro

nextgeniuspro commented Feb 4, 2025

Copy link
Copy Markdown
Owner

Also: would you mind creating a new release/tag

I don't think it needed for this lib. I don't plan to make big changes in near future, only bug fixes and small improvements

Thank you very much

@psyinf

psyinf commented Feb 5, 2025

Copy link
Copy Markdown
Contributor Author

Also: would you mind creating a new release/tag

I don't think it needed for this lib. I don't plan to make big changes in near future, only bug fixes and small improvements

Thank you very much

While not being needed it is handy for users of package managers such as CPM, conan or even plain CMake fetch_content, as it is "nicer" to address a library version by a tag rather than by a hash. E.g.

CPMAddPackage(
    NAME vfspp
    GITHUB_REPOSITORY nextgeniuspro/vfspp
    GIT_TAG v0.1.0
)

@nextgeniuspro

Copy link
Copy Markdown
Owner

Huh, ok, never though in this way. Probably you're right

@nextgeniuspro

Copy link
Copy Markdown
Owner

Added one release

@psyinf

psyinf commented Feb 6, 2025

Copy link
Copy Markdown
Contributor Author

Thank you!

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.

2 participants