Installed headers are a mixed blessing on Linux (developers have to keep track of which copy to modify) but overall probably a benefit. On Windows it seems the balance tips the other way; hence this project. End result will be to not install headers on Windows in the context of GlastRelease.
What needs to be done
Several files in the SConsFiles package need relatively small changes. Most are backwards-compatible and can be tagged early on. The final one, disabling the install of headers, should be in a new tag, made at the end of all other changes. Most other packages will also need changes to insure that, in all cases, headers belonging to one package will be accessible to other packages which need them. About 80 or the roughly 100 GR packages need changes of one sort or another. Packages which don't have any public headers can be left alone. There are 3 kinds of changes that need to be made, depending on characteristics of the package.
- Package xxx builds a shared library and has public headers. Need to add a couple lines to xxxLib.py (the file which gets invoked for the environment building xxx's library and also by other environements, building targets depending on that library) so that packages depending on xxx will get proper thing added to their include path
- Package xxx builds a static library. Here, since the library does not depend on other package libraries, components have to be added "by hand" to the include path. These changes, varying somewhat on a per-package basis, also go in xxxLib.py.
- Anything needing access to headers from the enums package. enums doesn't have a library. When headers are installed, there is no need for explicit action to get access to enums' headers. If they're not, packages needing those headers must add an entry to their include path.
The work is summarized below.
Package |
Work |
Status |
|---|---|---|
SConsFiles |
Add tool findPkgPath; modify msvs.py, addLinkDeps.py, SConstruct |
Committed and tagged except for addLinkDeps |
AcdRecon |
type 1 |
|
AcdUtil |
type 1 |
|
AdfEvent |
type 2 |
|
AnalysisNtuple |
type 1 |
|
AncillaryDataEvent |
type 2 |
|
AncillaryDataUtil |
type 2 |
|
astro |
type 1 |
|
CalibData |
type 1 |
|
calibRootData |
type 1 |
|
CalibSvc |
type 1 |
|
calibTkrUtil |
type 1 |
|
calibUtil |
type 1 |
|
CalRecon |
type 1 |
|
CalUtil |
type 1 |
|
CalXtalResponse |
type 1 |
|
celestialSources |
type 2 |
|
c/EarthPhenom |
type 2 |
|
c/eblAtten |
type 2 |
|
c/genericSources |
type 2 |
|
c/GRB |
type 2 |
|
c/GRBobs |
type 2 |
|
c/GRBtemplate |
type 2 |
|
c/microQuasar |
type 2 |
|
c/Pulsar |
type 2 |
|
c/SpectObj |
type 2 |
|
CHS/eventFile |
type 1 |
|
classifier |
type 2 |
|
commonRootData |
type 1 |
|
configData |
type 1 |
|
ConfigSvc |
type 1 |
|
detCheck |
type 2 |
|
detModel |
type 2 |
|
digiRootData |
type 1 |
|
EbfWriter |
type 1 |
|
embed_python |
type 2 |
|
Event |
type 1 |
|
evtUtils |
type 1 |
|
facilities |
type 1 |
done; not committed |
fitsGen |
type 2 |
|
flux |
type 2 |
|
FluxSvc |
type 1 |
|
G4Generator |
type 1 |
|
gcrSelectRootData |
type 1 |
|
geometry |
type 2 |
|
geomrep |
type 2 |
|
GlastClassify |
types 1, 2 |
|
GlastSvc |
type 1 |
|
gui |
type 2 |
|
GuiSvc |
type 1 |
|
HepRepSvc |
type 1 |
|
idents |
type 2 |
done, not committed |
LdfEvent |
type 1,3 |
|
ldfReader |
type 1 |
|
lsfData |
type 1 |
|
mcRootData |
type 1 |
|
mootCore |
type 1 |
|
MootSvc |
type 1 |
|
ntupleWriterSvc |
type 1 |
|
OnboardFilter |
type 1 |
|
OnboardFilterTds |
type 1 |
|
Overlay |
type 1 |
|
OverlayEvent |
type 1 |
|
overlayrootData |
type 1 |
|
rdbModel |
type 1 |
|
reconRootData |
type 1 |
|
RootConvert |
type 1 |
|
RootIo |
type 1 |
|
rootUtil |
type 1 |
|
tip |
type 2 |
|
TkrRecon |
type 1 |
|
TkrUtil |
type 1 |
|
Trigger |
type 1 |
|
xmlBase |
type 1 |
|
xmlUtil |
type 1 |
|
SConsFiles |
mod to registerTargets to not install headers |
done, tested. Not committed |