extractZipFile problem

[David Healey]

@Lindon Excellent, let us know your results. I need this feature too, in fact I've already implemented it but I didn't test it on Windows and MacOS yet, I might try this afternoon if I have some time.

[Lindon]

@d-healey sadly the simple "set in my window for all users" - doesnt work...we need a mechanism to define our app to open in admin mode...

[Lindon]

looks like it could be a setting in the manifest file in the Linker...

https://stackoverflow.com/questions/19617955/c-run-program-as-administrator

[David Healey]

@Lindon Working on a test project now, I'll report my results.

[David Healey]

Something I just saw in the docs which might be of interest but isn't a solution to the problem:

This method will extract a standard ZIP file (without password protection) to the given target directory (which can be either a file path String or a File object).

So you don't need to use FileSystem.fromAbsolutePath(), you can just pass in the string.

[David Healey]

I can confirm the issue on my system, can't extract to anywhere on the C drive. I'll keep investigating.

I've made a simple little test application for anyone who wants to play around with this. A test zip file is included in the project folder.

zipExtractor.zip

[David Healey]

@Lindon Yep you're correct. Add /MANIFESTUAC:level='requireAdministrator' to the linker flags section in Projucer and when your app starts you'll be prompted to give permission. Not sure how this will affect plugins, I'm going to test it now.

[Lindon]

@d-healey yeah I did this the other way(via VisualStudio)

and this seems to work too...

[David Healey]

@Lindon I tested in a plugin and it doesn't work. I assume that the plugin takes on the same privilege level as the host.

I'm going to test on MacOS now.

[Lindon]

@d-healey said in extractZipFile problem:

@Lindon I tested in a plugin and it doesn't work. I assume that the plugin takes on the same privilege level as the host.

I'm going to test on MacOS now.

-- test a stand alone on MacOS? yes? say yes, go on say yes...

[Lindon]

@d-healey said in extractZipFile problem:

@Lindon I tested in a plugin and it doesn't work. I assume that the plugin takes on the same privilege level as the host.

I think this may well be why NI Native Access and Arturia Software Centre etc. are stand alone programs only....they wont work as plugins because of this permissions issue...

[David Healey]

@Lindon

-- test a stand alone on MacOS? yes? say yes, go on say yes...

Yes. Just tested and it works fine, no permission issues. At least on my system.

I'm going to add that linker flag to my HISE fork's standalone template so it will automatically be added every time I export a standalone project.

[Lindon]

@d-healey - yeah I think MacOS is a bit more permissive than Windows - no surprise - it gets hacked a lot less.. but good to know, on to the next problem - well done.

[David Healey]

I've added a note about this to the docs - https://github.com/christophhart/hise_documentation/commit/0ced485944bc94c2bade19ab6a037d10c6db4c04

[Christoph Hart]

@d-healey yes but please don‘t make this the default - most people won‘t need admin access and it will look weird for a random eq plugin to ask for write access to the system folders.

[David Healey]

@Christoph-Hart I'm just making it the default on my fork, I'm not going to make a pull request. It's only in the standalone template, not the plugin template. At the moment everything I'm doing is with my standalone downloader so it's more convenient for me to have it as the default, I might change it in the future - or better yet use cmake ;)

[Christoph Hart]

@d-healey the proper solution would be to add a checkbox in the hise Project settings and then it will add those lines to the exporter template.

[David Healey]

@Christoph-Hart Ooo that's a good idea

[David Healey]

Looking at adding this now. Would an extra definition be better than a checkbox since it only applies to Windows?

[David Healey]

Ah I see why extra definitions aren't suitable. I'll just add a checkbox.