diff options
| author | Christian Breunig <christian@breunig.cc> | 2026-08-30 11:43:37 +0000 |
|---|---|---|
| committer | Christian Breunig <christian@breunig.cc> | 2026-08-30 11:43:37 +0000 |
| commit | a8da48e1d8a78b4ba3e543658795a194efb9ae27 (patch) | |
| tree | 54a0caf0ecda95e32c0e0213f44a54c18bbb84da /src/migration-scripts | |
| parent | 0e8c5e02f8b17b7b7a7905ec2b3057359edd02d6 (diff) | |
| download | vyos-1x-a8da48e1d8a78b4ba3e543658795a194efb9ae27.tar.gz vyos-1x-a8da48e1d8a78b4ba3e543658795a194efb9ae27.zip | |
debian: T9261: warn when byte-compilation fails instead of hiding it
The compileall call for the XML reference cache ended in '|| true', so a real
failure left no trace at all. Report it on stderr instead, and give py3compile
the same treatment so the two adjacent calls behave alike.
Deliberately not fatal. Bytecode is an optimisation: if it cannot be built the
CLI is slower but entirely correct, so aborting the installation - and with it
any image build - would be a worse outcome than a slow shell. This postinst has
no 'set -e' and contains no exit statements, so a failing command here has never
been treated as fatal; adding one now would make byte-compilation the single
exception in the file.
Note that compileall exits 0 for a missing directory and only returns nonzero on
an actual compilation error, so the warning fires on the case worth knowing
about and stays quiet otherwise.
Diffstat (limited to 'src/migration-scripts')
0 files changed, 0 insertions, 0 deletions
