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

5.3.28 tag problem (same than 5.3.21) #8

Open
Neustradamus opened this issue Oct 29, 2022 · 7 comments
Open

5.3.28 tag problem (same than 5.3.21) #8

Neustradamus opened this issue Oct 29, 2022 · 7 comments

Comments

@Neustradamus
Copy link

@gburd: There is a tag problem for 5.3.28:

Currently :

Can you add the 5.3.28 code to do not have only 5.3.21.

Can you add the last code in master and create the good tag?

Thanks in advance.

@greenkalx
Copy link

My vote for this as well.
These folders are missing from the repo: build_android, build_vxworks, build_wince, build_windows
But they are available in the 5.3.28 release: https://github.com/berkeleydb/libdb/releases/download/v5.3.28/db-5.3.28.tar.gz

I am compiling with Visual Studio 2022, there are some name collisions with Windows atomic:
C:\Program Files\Microsoft Visual Studio\2022\Professional\VC\Tools\MSVC\14.34.31933\include\atomic
I can provide a patch to rename some macros to avoid this.
Note these compile issues are not present for Visual Studio 2017.

Regards

@gburd
Copy link
Collaborator

gburd commented Dec 14, 2022 via email

@greenkalx
Copy link

I think we need to have 5.3.28 first, because my VS2022 patch modifies db.h in build_windows, and seven more files.
Want a 5.3.28 pull request first?

@greenkalx
Copy link

greenkalx commented Dec 15, 2022

I tried merging all the files from the release tarball, but it's 9000+ files and my git Fork app is spinning all night when I push.
I would try again, breaking it into two, one commit with just 'doc' folder of ~5000 files.

Is it enough to create a local branch and push for the branch to be created on github?
Or I need to fork the repo?
I'm coming from a dev environment where we create a remote branch from Jira. Then push to that branch and make a PR.

@greenkalx
Copy link

I also wasn't sure about the license and which files it covers for this repo.
In the tarball, there is a LICENSE file with an Oracle and NASM license.
In this repo, there is a LEGAL file with a Sleepycat license.

@Neustradamus
Copy link
Author

There is a problem of tag (5.3.21 = 5.3.28) here and the last official build with Sleepycat license is 6.0.19:

@greenkalx
Copy link

greenkalx commented Dec 22, 2022

The patch I'm using to compile with VS2022:
0006-vs2022_avoid_name_collisions.patch
This applies on the 5.3.28 release, not the repo which needs updating from 5.3.21 as I wrote above.

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

No branches or pull requests

3 participants