Skip to content
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

Install scripts don't fail if compilation fails #268

Open
lucic71 opened this issue Feb 10, 2023 · 0 comments
Open

Install scripts don't fail if compilation fails #268

lucic71 opened this issue Feb 10, 2023 · 0 comments

Comments

@lucic71
Copy link

lucic71 commented Feb 10, 2023

Some install.sh scripts are formed of two steps: first they compile the benchmark and then they run some decompressing. Take the example of git/aom-av1-1.1.0 (comments mine):

#!/bin/sh

## compilation step
rm -rf aom-master
git clone https://aomedia.googlesource.com/aom aom-master
mkdir aom-master/build
cd aom-master/build
cmake ..
make -j $NUM_CPU_CORES
echo $? > ~/install-exit-status
cd ~

## decompress step
7z x Bosphorus_1920x1080_120fps_420_8bit_YUV_Y4M.7z

echo "#!/bin/sh
./aom-master/build/aomenc -v --rt --threads=\$NUM_CPU_CORES --tile-columns=2 --limit=20 -o test.av1 Bosphorus_1920x1080_120fps_420_8bit_YUV.y4m > 1.log 2>&1
echo \$? > ~/test-exit-status
sed \$'s/[^[:print:]\t]/\\n/g' 1.log > \$LOG_FILE" > aom-av1
chmod +x aom-av1

However the script does not fail when the compilation fails and it will run the decompressing for no good reason, wasting time and CPU cycles.

What's my use case? I'm trying to see if some test profiles respect the CC/CXX variables set in the environment before calling something as:

phoronix-test-suite debug-install

I set CC=CXX=/garbage and clearly the compilation step fails but then the decompressing stars and takes an annoying amount of time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant