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

CPACK_WIX_ACL #1

Closed
ulrichard opened this issue Sep 5, 2014 · 4 comments
Closed

CPACK_WIX_ACL #1

ulrichard opened this issue Sep 5, 2014 · 4 comments

Comments

@ulrichard
Copy link
Contributor

Hi nils,

I wanted to test http://public.kitware.com/Bug/view.php?id=14924 earlier, but only now got to it.
I didn't get it to work so far. I didn't start debugging it yet, but maybe you see the problem before I have to.
This is what I tried:

INSTALL(DIRECTORY source/
DESTINATION "/"
COMPONENT app
PATTERN "Database/*.zip" EXCLUDE
)
IF(${CMAKE_VERSION} VERSION_GREATER 3.0.1)
SET_PROPERTY(INSTALL source/ PROPERTY CPACK_WIX_ACL Everyone=GenericAll)

INSTALL(FILES bergsteiger.JPG 
    DESTINATION "test"  
    COMPONENT test
)
SET_PROPERTY(INSTALL bergsteiger.JPG PROPERTY CPACK_WIX_ACL Everyone=GenericAll)

ENDIF()

This results in
'



'

@ngladitz
Copy link
Owner

ngladitz commented Sep 5, 2014

Install properties only work on files; not directories.
You also have to include the destination directory in your path.
I've got a working example of it in use here: https://github.com/ngladitz/cmake-wix-testsuite/blob/master/properties/CMakeLists.txt

@ulrichard
Copy link
Contributor Author

Thanks for the example.
I needed to change it to :
INSTALL(FILES bergsteiger.JPG
DESTINATION "test"
COMPONENT test
)
SET_PROPERTY(INSTALL test/bergsteiger.JPG
PROPERTY CPACK_WIX_ACL "Everyone=GenericAll"
)

But what I would really need is to set that on the directory, so that the installed application can create and populate sub-directories.
Looking at the code and documentation (http://ifw.podsvirov.pro/cmake/doc/sphinx/manual/cmake-properties.7.html#properties-on-installed-files) it looks like this is not supported by cpack, and is thus not really an option to implement in the WiX generator.
So, I guess I'll have to stick to the custom action for the time being.

@ngladitz
Copy link
Owner

ngladitz commented Sep 8, 2014

It isn't currently implemented but it might be possible to implement.
If you think it should be feel free to open an issue on the cmake tracker.

@ulrichard
Copy link
Contributor Author

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

2 participants