Hi Christoph,

thanks for your bug report.

> the build process for the datatables.js package uses static filenames
> in /tmp/, among them /tmp/closure_error.log which also does not get
> removed.

I see, you are absolutely right! There are definitely better ways to do
this on shared systems such as buildd machines.

> The much worse thing: The build does *not* catch that situation. Instead,
> the package is happily built with zero-sized files.

Ouch.

> The patch attached adds the usage of a random temporary directory that
> is cleaned up upon exit. Also the make.sh script now uses errexit. This
> should catch all unexpected errors during execution.

True. This looks like a much cleaner solution and is quite similar to
how I write most of my my autopkgtest scripts.

> According to diffoscope, the created binary packages are bitwise
> identical.

Sounds good, thanks for checking! I have forwarded the report to
upstream and also mentioned your patch.

>     Christoph, do not apply as-is
                 ^^^^^^^^^^^^^^^^^^

Do you mean 'do not apply the patch as-is'? If so, can you elaborate
please? Of course I understand that it's better to have upstream come up
with their own solution, but that doesn't mean that your patch isn't
helpful ;)

I'm not really sure about the 'grep "_buildInclude('"
DataTables.js.build >&2' line in make.sh though -- is it really
required or just a debugging relic?

Cheers
Sascha

Attachment: signature.asc
Description: OpenPGP digital signature

-- 
Pkg-javascript-devel mailing list
Pkg-javascript-devel@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-javascript-devel

Reply via email to