On 12/03/2009 05:08 AM, Allan McRae wrote:
Cedric Staniewski wrote:
When makepkg exits in create_srcpackage(), the (temporary) srclinks directory is left behind.
Signed-off-by: Cedric Staniewski <cedric@gmx.ca> --- scripts/makepkg.sh.in | 1 + 1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in index ceaa8a6..b618881 100644 --- a/scripts/makepkg.sh.in +++ b/scripts/makepkg.sh.in @@ -118,6 +118,7 @@ trap_exit() { echo error "$@" fi + [[ -n $srclinks ]] && rm -rf "$srclinks" exit 1 }
I am not sure about this as if the source package creation fails, I would like to be able to look in the directory to track down why. Much like how "-c" does not clear the source directory on a build failure. So my initial reaction is a -1.
As an aside, did you actually encounter an error during source package creation or just note that could be left behind from the code?
I actually did not encounter an error but aborted via ctrl+c. Keeping this folder is useless in my opinion as there are error messages for pretty much everything and the three different external commands which are used (mkdir, ln, bsdtar) usually throw meaningful error messages, too. Aside from that this folder only contains smylinks anyway and I doubt they are useful for tracking down errors. Another alternative would be introducing a new cleanup function which is only run on SIGINT and moving this line there, though.