Hi Alexander,
I would prefer to have an install_bundles macro because it is more
clear what the macro will actually do. At some point I actually thought
about adding something like an install_drivers macro for driver-bundles
(according to the device access spec.) to get them automatically
deployed to a drivers folder instead of the bundles folder. This could
be easily done with the install_bundles solution but would need a new
argument for the bundle solution.
Regards,
Bjoern
Am 2013-09-29 16:27, schrieb Alexander Broekhuis:
Hi all,
I've been working on this a bit and now have 2 options. I'd like to
know
what others think.
The first option is to include all the installation stuff in the
"bundle"
macro. For example:
bundle(shell SOURCES ... INSTALL HEADERS ..../header.h RESOURCES
..../files)
The following arguments are new:
INSTALL: marks this bundle to be installed.
HEADERS: optional headers to be installed in
.../include/celix/bundle_name
RESOURCES: optional resources to be installed in
../share/celix/bundle_name
The second option is a separate macro for all these arguments
install_bundle(shell
HEADERS
..../header.h
RESOURCES
..../files
)
This macro doesn't need an INSTALL option, the use of it marks the
bundle
to be install. The other arguments are used the same as above.
Maybe there are some missing arguments, those can easily be added.
For now
my question is which one do others on this list like?
2013/9/27 Alexander Broekhuis <[email protected]>
2013/9/27 Björn Petri <[email protected]>
Hi All,
I just can agree - I think that's quite a nice idea. But ,aybe this
could
also include the export of all the according public/include
header-files?
At the moment not all bundles export their header-files by default
(e.g.
rsa), which makes it difficult to setup a build-infrastructure next
to
celix.
That should also be possible with the new updates. Any missing files
are
bugs. I'll take a look into it as well. If I have pushed the update
and
there are still missing files, feel free to raise a bug for those.
--
Met vriendelijke groet,
Alexander Broekhuis
