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

[FEATURE] - Use NanaZip fork instead of 7-Zip #1021

Open
agowa opened this issue Jan 18, 2024 · 13 comments
Open

[FEATURE] - Use NanaZip fork instead of 7-Zip #1021

agowa opened this issue Jan 18, 2024 · 13 comments
Labels
enhancement New feature or request proposal This is put forwarded for consideration

Comments

@agowa
Copy link

agowa commented Jan 18, 2024

What is your feature request regarding to?

Atlas Playbook

Is your feature request related to a problem? Please describe.

Consider using the NanaZip fork of 7-Zip instead of 7-Zip itself as it has better integration into windows 11.
E.g. not needing to modify the path environment variable in order to use 7z.exe from the cli (because of it registering itself within "Apps > Advanced app settings > App execution aliases").

Describe the solution you would like.

Instead of installing 7-Zip through the playbook install NanaZip

Describe alternatives you have considered.

does not apply

Additional context.

No response

@agowa agowa added the enhancement New feature or request label Jan 18, 2024
@n00b69
Copy link

n00b69 commented Jan 18, 2024

+1

Also, not sure if still applicable, but in the past Nanazip Preview worked better on win11 than the regular one. I have only ever used the Preview version, so I cannot confirm this.

@Revival8697
Copy link

-1

Consider to NOT use NanaZip till a stable release is out.

@JayXTQ
Copy link
Member

JayXTQ commented Jan 18, 2024

let's not have a debate about this topic. if there is no stable release, this issue will be put on hold until there is a stable release. closing for now until then.

@JayXTQ JayXTQ closed this as not planned Won't fix, can't repro, duplicate, stale Jan 18, 2024
@agowa
Copy link
Author

agowa commented Jan 19, 2024

@JayXTQ But there is a stable release https://apps.microsoft.com/detail/9N8G7TSCL18R?hl=en-us&gl=US

@JayXTQ
Copy link
Member

JayXTQ commented Jan 19, 2024

just because it's on ms store doesn't mean it's stable lol. until there is valid evidence that it is stable this will be closed. or if it's not needed in atlas, which it isn't really as 7z works fine

@agowa
Copy link
Author

agowa commented Jan 19, 2024

7z is neither added to the path, nor does it register itself for an app execution alias, so "7z works fine" is debatable.
Also NanaZip has a bunch of improvements just look at the list either on the store page or their GitHub. Like it supporting additional compression algorithms, archive types, better dpi awareness for multi monitor setups, additional hashing algorithms, improved security, ...

Also what do you want as "prove" that it isn't unstable if this isn't enough?:
This is the stable version: https://apps.microsoft.com/detail/9N8G7TSCL18R?hl=en-us&gl=US
And this is the preview version: https://apps.microsoft.com/detail/9NZL0LRP1BNL?hl=en-us&gl=US
And this is the projects GitHub page: https://github.com/M2Team/NanaZip

@JayXTQ
Copy link
Member

JayXTQ commented Jan 19, 2024

none of those benefits are needed in atlas and it doesn't need to be added to path for it to work for the average user. you forget this is not something for developers but for gamers.

@Revival8697
Copy link

Revival8697 commented Jan 19, 2024

But there is a stable release

improved security

M2Team/NanaZip#330, M2Team/NanaZip#331. The stable build of NanaZip is severely outdated.

Also what do you want as "prove" that it isn't unstable if this isn't enough?

v3 is in Preview, we shouldn't ship Preview/Beta/Unstable builds to users.

@Ast3risk-ops
Copy link

I had no idea about those and I used the stable build of nanazip when on windows 💀

@agowa
Copy link
Author

agowa commented Jan 20, 2024

Same, wasn't aware of these ones.

@francisco-lafe
Copy link

I'm not sure I'm understanding why NanaZip 2.0 (stable) is not being considered as an option.

@Ast3risk-ops
Copy link

I'm not sure I'm understanding why NanaZip 2.0 (stable) is not being considered as an option.

Security issues and it's not actually stable and 7zip works just fine.

@he3als he3als added the proposal This is put forwarded for consideration label Jan 24, 2024
@he3als he3als reopened this Jan 24, 2024
@he3als
Copy link
Member

he3als commented Jan 24, 2024

You can argue security issues, but 7-Zip doesn't automatically update or notify the users of updates. Once NanaZip gets more developed, it should be fundamentally more secure, as NanaZip is installed and updated via the Store.

Currently though, I don't think that NanaZip is in a developed enough state to make it worth using over 7-Zip, and the process of automatic installation inside the playbook will be a bit more difficult due to it being AppX.

I'll keep this open to track it's development.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request proposal This is put forwarded for consideration
Projects
None yet
Development

No branches or pull requests

7 participants