-
Notifications
You must be signed in to change notification settings - Fork 39
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
Improved module dependencies #36
base: master
Are you sure you want to change the base?
Improved module dependencies #36
Conversation
Are these changes all SLAC-specific? Certainly that long list of modules in My build of iocStats-3.1.16 has installed the The switch to the
TBH since I suspect nobody outside of SLAC can properly review or test these changes I think the rest of us would prefer that all the SLAC-specific files be moved into a separate SLAC-only module. Non-SLAC maintainers can't do much work on this module as it exists right now since we can't tell if we've broken your parts. |
I would assume no other sites are currently using iocReleaseCreateDb.py as the prior version only read $TOP/RELEASE_SITE and $TOP/configure/RELEASE and only handled SLAC style ASYN_MODULE_VERSION release versions. As part of an effort to merge the accelerator version of iocAdmin w/ the photon version (where we weren't using iocReleaseCreateDb.py), I made the python script more robust and able to work in both SLAC environments. I think this version will likely work for most other sites, but have no feedback yet as to whether or not there is any interest from other sites. If you add the IOC_RELEASE_DB = iocRelease.db line to your IOC it should produce iocRelease.db, a collection of stringin PVs named Re pkgNamesToMacroNames.py, I've removed SLAC specific comments. Any sites are welcome to contribute macro names to the list. It's just used for cosmetic purposes so the DESC field can be populated with the actual module name instead of a macro like ADCORE. Re installation, if we want it to work from the IOCADMIN INSTALL_LOCATION we need to install it when the iocStats module is built, as we don't know yet which IOC's will define IOCADMIN and IOC_RELEASE_DB. I also made the python2 default to python. Re splitting out SLAC specific files to other modules, it appears that other sites are using iocAdmin/Db. I have no objection to moving iocAdmin/srcDisplay/*.edl to op/edl along w/ Mark's other UI files. The srcRestore directory just has some old channelWatcher config files that may not even be used by any SLAC IOCs. SLAC isn't deploying the master branch anyway, so there's some flexibility re how we handle this. As @steffallison has retired, Ernest Williams suggested that @kukhee and/or myself might get granted write access to this github repo. |
Hi Bruce, to answer your last point, @kukhee should be able to write to this repository as he is a member of the @epics-modules/iocstats-developers team and I believe has been since he was nominated as its maintainer by Ernest quite some time ago. I'd be happy to add you to that team too, as you're the one doing work on the module. I haven't had a chance to look at your latest changes myself, and I probably won't now until after ICALEPCS. |
Needed so LCLS iocReleaseCreateDb.py, written for python 2.6 on RHEL6, can be used for PCDS RHEL5 IOC buids using python 2.4.
Also bump up max # of dependencies from 20 to 30.
Also don't generate ioc version in iocRelease.db as it's already available via :APP_DIR
1e66cf1
to
628d7d3
Compare
FYI - I'm retiring from SLAC and don't have any strong opinions re whether or not this pull request gets accepted, so I'll leave that up to Simon and Kukhee to work out. Cheers, Bruce |
Replaced original SLAC specific iocReleaseCreateDb.py w/ improved python module dependency generation that handles more variations on RELEASE files included following nested includes.
This version should work for other sites as well.
To test, add this line in your IOC *App/Db/Makefile:
IOCRELEASE_DB += iocRelease.db