Page MenuHomeFreeBSD

elfcopy: stop leaving temporary files on error
ClosedPublic

Authored by crees on Feb 13 2021, 4:33 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Oct 23, 8:58 AM
Unknown Object (File)
Oct 7 2024, 9:08 AM
Unknown Object (File)
Oct 2 2024, 5:03 AM
Unknown Object (File)
Oct 1 2024, 12:49 PM
Unknown Object (File)
Oct 1 2024, 4:44 AM
Unknown Object (File)
Oct 1 2024, 2:38 AM
Unknown Object (File)
Sep 30 2024, 4:19 AM
Unknown Object (File)
Sep 24 2024, 11:12 AM
Subscribers

Details

Summary

Temporary files were not cleaned up, resulting in $TMPDIR or even
the current directory becoming littered with ecp.* files.

This happened with error and even sometimes on success!

Test Plan

make buildworld

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 37016
Build 33905: arc lint + arc unit

Event Timeline

crees requested review of this revision.Feb 13 2021, 4:33 PM

Hold for a bit- it errs on the verbose side, so in the test suite we get unnecessary stderr output. Will add extra checks.

elftoolchain: stop leaving tempfiles on error

Temporary files were not cleaned up, resulting in $TMPDIR or even
the current directory becoming littered with ecp.* files.

This happened with error and even sometimes on success!

Obtained from: https://sourceforge.net/p/elftoolchain/code/3918/

I think this should be ready to go- upstream has accepted it. Are you happy with it?

Ah sorry, totally forgotten about this. It seems to be fine, let's get it in then.

This revision is now accepted and ready to land.Mar 3 2021, 4:42 PM
This revision was automatically updated to reflect the committed changes.

Is there a reason why this has not been back ported to stable/12? Here are two issues suffering from the lack: