archivematica / Issues

Issues repository for the Archivematica project
GNU Affero General Public License v3.0
16 stars 1 forks source link

Validation Command stderr output missing #440

Open ghost opened 5 years ago

ghost commented 5 years ago

Place in Dashboard: Validation > Standard error (stderr) Place in Code: lib/clientScripts/validate_file.py (around 145)

Expected behaviour ... failed with exit status 1; stderr: here are details to this error ...

Current behaviour ... failed with exit status 1;

Steps to reproduce Step Validation fails -> Details-Symbol -> See Text in "Standard error (stderr)"

Additional Infos I cross-checked the previous commit with the latest changes since in am-1.7 i did not have this problem and it appears that the last line of the previous Codepart:

print('Command {description} failed with exit status {status};'
                  ' stderr:'.format(
                      description=rule.command.description,
                      status=exitstatus),
                  stderr, file=sys.stderr)

has been removed completly, compared to the latest Codepart:

self.job.print_error(
                'Command {description} failed with exit status {status};'
                ' stderr:'.format(description=rule.command.description,
                                  status=exitstatus))

, which always results in stderr: nothing

Your environment (version of Archivematica, OS version, etc) 1.8.1 / Archivematica-1.8 Distributor ID: Ubuntu Description: Ubuntu 16.04.5 LTS Release: 16.04 Codename: xenial


For Artefactual use: Please make sure these steps are taken before moving this issue from Review to Verified in Waffle:

sromkey commented 5 years ago

HI @VinzenzBauer , thanks for filing. Just a few follow up questions:

ghost commented 5 years ago

Standard streams Standard output (stdout)

Running Validate using CheckitTiff Running Validate using CheckitTiff_upcoming

Standard error (stderr)

Command Validate using CheckitTiff failed with exit status 1; stderr: Command Validate using CheckitTiff_upcoming failed with exit status 1; stderr:

ghost commented 5 years ago

Took me a while, i put the error-out to eventOutcomeDetailNote which works almost like the print >> sys.stderr, in AM 1.7.

This issue does not have priority to me anymore but i'd still like to see that the sys.stderr will be shown again. Thank you.