-
Notifications
You must be signed in to change notification settings - Fork 11
Open
Description
In the gemctp7user
tools repo we rely on being able to specify a release version and getting the corresponding files.
Currently, this is muddled by mixing different era files within a given "release".
While understandable from a pre-release perspective, it would be best if when a release becomes "final" it always has the same convention.
The expectation (based on previous releases) is that this will always work:
then
echo "CTP7 firmware fw/gem_ctp7_v${ctp7fw//./_}_GBT.bit missing, downloading"
echo "wget https://github.com/evka85/GEM_AMC/releases/download/v${ctp7fw}/gem_ctp7_v${ctp7fw//./_}_GBT.bit -O fw/gem_ctp7_v${ctp7fw//./_}_GBT.bit"
wget https://github.com/evka85/GEM_AMC/releases/download/v${ctp7fw}/gem_ctp7_v${ctp7fw//./_}_GBT.bit -O fw/gem_ctp7_v${ctp7fw//./_}_GBT.bit
fi
echo "ln -sf gem_ctp7_v${ctp7fw//./_}_GBT.bit fw/gem_ctp7.bit"
ln -sf gem_ctp7_v${ctp7fw//./_}_GBT.bit fw/gem_ctp7.bit
if [ ! -f "xml/gem_amc_top_${ctp7fw//./_}.xml" ]
then
echo "CTP7 firmware xml/gem_amc_top_${ctp7fw//./_}.xml missing, downloading"
echo "wget https://github.com/evka85/GEM_AMC/releases/download/v${ctp7fw}/address_table_v${ctp7fw//./_}_GBT.zip"
wget https://github.com/evka85/GEM_AMC/releases/download/v${ctp7fw}/address_table_v${ctp7fw//./_}_GBT.zip
echo "unzip address_table_v${ctp7fw//./_}_GBT.zip"
unzip address_table_v${ctp7fw//./_}_GBT.zip
echo "cp address_table_v${ctp7fw//./_}_GBT/gem_amc_top.xml xml/gem_amc_v${ctp7fw//./_}.xml"
cp address_table_v${ctp7fw//./_}_GBT/gem_amc_top.xml xml/gem_amc_v${ctp7fw//./_}.xml
echo "rm -rf address_table_v${ctp7fw//./_}_GBT"
rm -rf address_table_v${ctp7fw//./_}_GBT
fi
If some of this is obsolete (i.e.,, using the _GBT
suffix), that's fine, but the general structure needs to be consistent (see also cms-gem-daq-project/xhal#27)
Metadata
Metadata
Assignees
Labels
No labels