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

Add latest version to releases #281

Closed
Epictyphlosion opened this issue Nov 18, 2022 · 20 comments
Closed

Add latest version to releases #281

Epictyphlosion opened this issue Nov 18, 2022 · 20 comments

Comments

@Epictyphlosion
Copy link

Now that 7-zip ZS has been caught up with mainline 7-zip, I don't see any reason why version 22.01 shouldn't be in the releases page.

@mcmilk
Copy link
Owner

mcmilk commented Nov 18, 2022

The current build does not work thats why it is so.

@mshedsilegx
Copy link

Does this help ?
#274
Patch in: #274 (comment)

@mcmilk
Copy link
Owner

mcmilk commented Jan 15, 2023

I did a pre-release today ...

@mshedsilegx
Copy link

This is much appreciated.

@mcmilk
Copy link
Owner

mcmilk commented Jan 19, 2023

I don't know winget repo?

@mcmilk
Copy link
Owner

mcmilk commented Jan 20, 2023

I had no idea what winget is ... it look like some package manager for windows. I did't create some packet for it ?

@mcmilk
Copy link
Owner

mcmilk commented Jan 20, 2023

Now that 7-zip ZS has been caught up with mainline 7-zip, I don't see any reason why version 22.01 shouldn't be in the releases page.

DONE ;)

@mcmilk mcmilk closed this as completed Jan 20, 2023
@mcmilk
Copy link
Owner

mcmilk commented Apr 7, 2023

I am not ...

@Speed4Trade-Admin
Copy link

The problem with winget is the different version number:
22.01 ZS v1.5.5 R2
vs
22.01-v1.5.5-R2
If they don't match exactly, the update is shown again and again.
One version comes from the setup which is adding the version in the registry:
Computer\HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Windows\CurrentVersion\Uninstall\7-Zip-Zstandard

@mcmilk
Copy link
Owner

mcmilk commented Apr 9, 2023

All my releases are done this way:
19.01-v1.2.3-R1
20.04-v1.3.4-R1
22.01-v1.5.1-R1 22.01-v1.5.1-R3 22.01-v1.5.1-R3
22.01-v1.5.5-R1 22.01-v1.5.5-R2 22.01-v1.5.5-R3
And so on .... First number is 7-Zip, the second number Zstandard, in the end my own release number.

You can look into the tag history.

@thomass4t
Copy link

Could the header file "7zVersion.h" be the source of the problem?

#define MY_VERSION_NUMBERS "22.01 ZS v1.5.5 R2"

There is an additional "ZS"

@mcmilk
Copy link
Owner

mcmilk commented Apr 9, 2023

These are two sources.

This header file 7zVersion.h has since it's beginning the other naming - it's used for the about box and other things.
The tag is used for the source code at github.

This winget thing should always use the same source for it's comparison.

@thomass4t
Copy link

I think the mistake is located there:
https://github.com/microsoft/winget-pkgs/tree/master/manifests/m/mcmilk/7zip-zstd

I dont know where they pull the version number from.

@thomass4t
Copy link

Here is some information how it works:
microsoft/winget-cli#2613

There is also a github action available to publish new versions and thus don't rely on web scraping and similar stuff.

@vedantmgoyal9
Copy link

Hello, the automation gets the version from the GitHub releases tags with the starting v removed from it. So it is something like:
v22.01-v1.5.5-R2 - starting v => 22.01-v1.5.5-R2.

Also, yes, I do have a GitHub Action vedantmgoyal2009/winget-releaser which can be run as a CI job, if the maintainer is comfortable with.

@thomass4t
Copy link

I would recommend to either use a GitHub Action or just keep the versions in sync with the header file:
#define MY_VERSION_NUMBERS "22.01 ZS v1.5.5 R2"

@mcmilk
Copy link
Owner

mcmilk commented Apr 10, 2023

Hello, the automation gets the version from the GitHub releases tags with the starting v removed from it. So it is something like: v22.01-v1.5.5-R2 - starting v => 22.01-v1.5.5-R2.

This looks okay.

Also, yes, I do have a GitHub Action vedantmgoyal2009/winget-releaser which can be run as a CI job, if the maintainer is comfortable with.

Yes, I have no problem with that :-)

@mcmilk
Copy link
Owner

mcmilk commented Apr 10, 2023

The version string from the header can't be used for this then ...

@mcmilk
Copy link
Owner

mcmilk commented Apr 10, 2023

The string "22.01 ZS v1.5.4 R4" within the header is not technical ... it's a define I used for some about box.
@youk - please stop bugging me because auf this string. I don't want some - there ....
The winget package should no use this string ... thats it.

@youk
Copy link

youk commented Apr 10, 2023

You may do whatever you please. Nobody is "bugging" you. Ciao.

@mcmilk mcmilk closed this as completed Apr 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants