[Pkg-javascript-devel] Bug#902507: node-multiparty: FTBFS in buster/sid (TypeError: mime.lookup is not a function)

2018-10-06 Thread Petter Reinholdtsen
I had a closer look, and became more confused.  First some background
information: The mime library renamed lookup() to getType, while the
mime-types library in sid seem to still provide the lookup() method.

This build problemseem to be caused by a test depending on form-data.js in
the node-form-data package, which expect a mime library with the lookup()
function.  One way to fix this would be to convince form-data.js to use
the mime-types library, but I am unable to find out which library is
actually used and how it is pulled in.

A fix would be to change the code in node-form-data to use getType()
instead of lookup(), but it make more sense if a dependency of
node-form-data could be updated to reflect the changed requirement, to
version 2 of the node-mime library. The problem is that node-form-data
do not (build-)depend on any node-mime* library I can see.  What is
going on here?  What is the best fix for this?

In any case, I suspect this bug should be reassigned node-form-data or
split in two, one for node-multiparty and one for node-form-data.
But given that I have not been able to figure out exactly where
a mime* library is loaded, I do not know where the bug really is
located.

-- 
Happy hacking
Petter Reinholdtsen

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

[Pkg-javascript-devel] Bug#902507: node-multiparty: FTBFS in buster/sid (TypeError: mime.lookup is not a function)

2018-10-06 Thread Petter Reinholdtsen
The lookup() function in the mime library changed name to getType()
in the mime library version 2.  The dependency should be changed and
the name updated to solve this problem.

-- 
Happy hacking
Petter Reinholdtsen

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

[Pkg-javascript-devel] Bug#902507: node-multiparty: FTBFS in buster/sid (TypeError: mime.lookup is not a function)

2018-06-27 Thread Santiago Vila
Package: src:node-multiparty
Version: 4.0.0-1
Severity: serious
Tags: ftbfs

Dear maintainer:

I tried to build this package in buster with "dpkg-buildpackage -A"
but it failed:


[...]
 debian/rules build-indep
dh build-indep
dh: Compatibility levels before 9 are deprecated (level 8 in use)
   dh_update_autotools_config -i
   debian/rules override_dh_auto_test
make[1]: Entering directory '/<>'
nodejs test/test.js
Fixture tests:
content-type/charset.http...(node:20301) [DEP0022] DeprecationWarning: 
os.tmpDir() is deprecated. Use os.tmpdir() instead.
OK
content-type/charset-last.http...OK
content-type/custom.http...OK
content-type/custom-equal-sign.http...OK

[... snipped ...]

special-chars-in-filename/osx-firefox-3.6.http...OK
special-chars-in-filename/osx-safari-5.http...OK
special-chars-in-filename/xp-chrome-12.http...OK
special-chars-in-filename/xp-ie-7.http...OK
special-chars-in-filename/xp-ie-8.http...OK
special-chars-in-filename/xp-safari-5.http...OK

Standalone tests:
test-chunked...(node:20311) [DEP0022] DeprecationWarning: os.tmpDir() is 
deprecated. Use os.tmpdir() instead.
OK
test-connection-aborted-closed...(node:20325) [DEP0022] DeprecationWarning: 
os.tmpDir() is deprecated. Use os.tmpdir() instead.
OK
test-connection-aborted...(node:20335) [DEP0022] DeprecationWarning: 
os.tmpDir() is deprecated. Use os.tmpdir() instead.
OK
test-content-transfer-encoding...(node:20345) [DEP0022] DeprecationWarning: 
os.tmpDir() is deprecated. Use os.tmpdir() instead.
OK
test-emit-order.../usr/lib/nodejs/form-data/form_data.js:130
'Content-Type: ' +  (options.contentType || 
mime.lookup(options.filename || value.path));
 ^

TypeError: mime.lookup is not a function
at FormData._multiPartHeader (/usr/lib/nodejs/form-data/form_data.js:130:58)
at FormData.append (/usr/lib/nodejs/form-data/form_data.js:39:21)
at Request.attach (/usr/lib/nodejs/superagent/lib/node/index.js:200:18)
at Server. 
(/<>/test/standalone/test-emit-order.js:51:7)
at Object.onceWrapper (events.js:313:30)
at emitNone (events.js:106:13)
at Server.emit (events.js:208:7)
at emitListeningNT (net.js:1382:10)
at _combinedTickCallback (internal/process/next_tick.js:135:11)
at process._tickCallback (internal/process/next_tick.js:180:9)
/<>/test/test.js:61
if (code) throw new Error("exited with code " + code);
  ^

Error: exited with code 1
at ChildProcess. (/<>/test/test.js:61:25)
at emitTwo (events.js:126:13)
at ChildProcess.emit (events.js:214:7)
at Process.ChildProcess._handle.onexit (internal/child_process.js:198:12)
debian/rules:11: recipe for target 'override_dh_auto_test' failed
make[1]: *** [override_dh_auto_test] Error 1
make[1]: Leaving directory '/<>'
debian/rules:8: recipe for target 'build-indep' failed
make: *** [build-indep] Error 2
dpkg-buildpackage: error: debian/rules build-indep subprocess returned exit 
status 2


The above is how the build ends in my autobuilder and it's not
necessarily the relevant part.

The failure is not, however, related to using dpkg-buildpackage -A,
as it also fails in reproducible builds:

https://tests.reproducible-builds.org/debian/rb-pkg/unstable/amd64/node-multiparty.html

If this is really a bug in one of the build-depends, please use reassign and 
affects,
so that this is still visible in the page for this package.

Thanks.

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