mirror of
https://github.com/slsdetectorgroup/slsDetectorPackage.git
synced 2025-06-04 00:50:42 +02:00
adresses review comments
This commit is contained in:
parent
9343e3c667
commit
6d2f34ef1d
@ -10,8 +10,11 @@ det_list=("ctbDetectorServer
|
||||
)
|
||||
usage="\nUsage: compileAllServers.sh [server|all(opt)] [update_api(opt)]. \n\tNo arguments mean all servers with 'developer' branch. \n\tupdate_api if true updates the api to version in VERSION file"
|
||||
|
||||
update_api=false
|
||||
target=clean
|
||||
update_api=true
|
||||
target=version
|
||||
|
||||
|
||||
|
||||
# arguments
|
||||
if [ $# -eq 0 ]; then
|
||||
# no argument, all servers
|
||||
@ -37,9 +40,8 @@ elif [ $# -eq 1 ] || [ $# -eq 2 ]; then
|
||||
|
||||
if [ $# -eq 2 ]; then
|
||||
update_api=$2
|
||||
if $update_api ; then
|
||||
target=version
|
||||
echo "updating api to $(cat ../VERSION)"
|
||||
if not $update_api ; then
|
||||
target=clean
|
||||
fi
|
||||
fi
|
||||
else
|
||||
@ -51,6 +53,9 @@ declare -a deterror=("OK" "OK" "OK" "OK" "OK" "OK")
|
||||
|
||||
echo -e "list is ${det[@]}"
|
||||
|
||||
if $update_api; then
|
||||
echo "updating api to $(cat ../VERSION)"
|
||||
fi
|
||||
# compile each server
|
||||
idet=0
|
||||
for i in ${det[@]}
|
||||
@ -67,7 +72,6 @@ do
|
||||
fi
|
||||
mv bin/$dir bin/$file
|
||||
git add -f bin/$file
|
||||
cp bin/$file /tftpboot/
|
||||
cd ..
|
||||
echo -e "\n\n"
|
||||
((++idet))
|
||||
|
@ -6,21 +6,25 @@ file="${dir}_developer"
|
||||
|
||||
usage="\nUsage: compileAllServers.sh [update_api(opt)]. \n\t update_api if true updates the api to version in VERSION file"
|
||||
|
||||
update_api=false
|
||||
target=clean
|
||||
update_api=true
|
||||
target=version
|
||||
|
||||
# arguments
|
||||
if [ $# -eq 1 ]; then
|
||||
update_api=$1
|
||||
if $update_api ; then
|
||||
target=version
|
||||
echo "updating api to $(cat ../VERSION)"
|
||||
if not $update_api ; then
|
||||
target=clean
|
||||
|
||||
fi
|
||||
elif [ ! $# -eq 0 ]; then
|
||||
echo -e "Only one optional argument allowed for update_api."
|
||||
return -1
|
||||
fi
|
||||
|
||||
if $update_api; then
|
||||
echo "updating api to $(cat ../VERSION)"
|
||||
fi
|
||||
|
||||
echo -e "Compiling $dir [$file]"
|
||||
cd $dir
|
||||
if make $target; then
|
||||
@ -31,7 +35,6 @@ fi
|
||||
|
||||
mv bin/$dir bin/$file
|
||||
git add -f bin/$file
|
||||
cp bin/$file /tftpboot/
|
||||
cd ..
|
||||
echo -e "\n\n"
|
||||
printf "Result:\t\t= %s\n" "${deterror}"
|
||||
|
@ -33,7 +33,6 @@ versioning:
|
||||
|
||||
$(PROGS): $(OBJS)
|
||||
# echo $(OBJS)
|
||||
cd $(current_dir)
|
||||
mkdir -p $(DESTDIR)
|
||||
$(CC) -o $@ $^ $(CFLAGS) $(LDLIBS)
|
||||
mv $(PROGS) $(DESTDIR)
|
||||
@ -41,7 +40,7 @@ $(PROGS): $(OBJS)
|
||||
rm $(main_src)*.o $(md5_dir)*.o
|
||||
|
||||
clean:
|
||||
cd $(current_dir) && rm -rf $(DESTDIR)/$(PROGS) *.o *.gdb $(main_src)*.o $(md5_dir)*.o
|
||||
rm -rf $(DESTDIR)/$(PROGS) *.o *.gdb $(main_src)*.o $(md5_dir)*.o
|
||||
|
||||
|
||||
|
||||
|
@ -11,7 +11,7 @@ class Version {
|
||||
private:
|
||||
std::string version_;
|
||||
std::string date_;
|
||||
inline static const std::string defaultBranch_[] = {"developer", "0.0.0"};
|
||||
inline static const std::string defaultVersion_[] = {"developer", "0.0.0"};
|
||||
|
||||
public:
|
||||
explicit Version(const std::string &s);
|
||||
|
@ -22,7 +22,7 @@ Version::Version(const std::string &s) {
|
||||
|
||||
bool Version::hasSemanticVersioning() const {
|
||||
|
||||
return (version_ != defaultBranch_[0]) && (version_ != defaultBranch_[1]);
|
||||
return (version_ != defaultVersion_[0]) && (version_ != defaultVersion_[1]);
|
||||
}
|
||||
|
||||
std::string Version::getVersion() const { return version_; }
|
||||
|
@ -20,7 +20,7 @@ VERSION_FILE = SCRIPT_DIR + "/VERSION"
|
||||
|
||||
parser = argparse.ArgumentParser(description = 'updates API version')
|
||||
parser.add_argument('api_module_name', choices=["APILIB", "APIRECEIVER", "APICTB", "APIGOTTHARD2", "APIMOENCH", "APIEIGER", "APIXILINXCTB", "APIJUNGFRAU", "APIMYTHEN3"], help = 'module name to change api version options are: ["APILIB", "APIRECEIVER", "APICTB", "APIGOTTHARD2", "APIMOENCH", "APIEIGER", "APIXILINXCTB", "APIJUNGFRAU", "APIMYTHEN3"]')
|
||||
parser.add_argument('api_dir', help = 'Relative or absolut path to the module code')
|
||||
parser.add_argument('api_dir', help = 'Relative or absolute path to the module code')
|
||||
|
||||
def update_api_file(new_api : str, api_module_name : str, api_file_name : str):
|
||||
|
||||
|
Loading…
x
Reference in New Issue
Block a user