We should not hardcode release-trunk-buster there as
mr7.5 will be branched soon and the code will not be valid.
Instead we should use the common approach to allow Jenkins
provide proper/release-specific apt files.
Change-Id: Ica6df944d3fa5c547b2677fcf7e6e128ebd74967
Removing nodejs-legacy as it is not availabe in buster,
also it should be unnecessary nowadays.
It was adde in commit f3e1d8e621
> TT#22402 Switch janus-admin Dockerfile to Debian stretch
> ...
> We have to install 'nodejs-legacy' otherwise /usr/bin/node
> is not available and test 'janus-admin-unit-test-docker' fails:
> ...
> + docker run --rm -i --entrypoint /bin/bash -v /var/lib/jenkins/workspace/janus-admin-unit-test-docker/source/:/code:ro \
> -v /var/lib/jenkins/workspace/janus-admin-unit-test-docker/results:/results:rw \
> docker.mgm.sipwise.com/janus-admin-stretch:I3370beedf30495497d01ec42eb292f90bce2b0a2 -c /code/t/testrunner
> ### Copying and moving files
> ################################################################################
> Finished main setup, now running tests ...
>
> > janus-admin@1.0.4 test /tmp/code
> > mocha -R spec --full-trace test/*-spec.js
>
> /usr/bin/env: 'node': No such file or directory
> npm ERR! Test failed. See above for more details.
Change-Id: I19465cfbca0cdfffe43293abffee18fac0741851
We have to install 'npm' explicitly here as it is no longer provided
by package 'nodejs' in Debian stretch.
We have to install 'nodejs-legacy' otherwise /usr/bin/node
is not available and test 'janus-admin-unit-test-docker' fails:
> + docker run --rm -i --entrypoint /bin/bash -v /var/lib/jenkins/workspace/janus-admin-unit-test-docker/source/:/code:ro \
> -v /var/lib/jenkins/workspace/janus-admin-unit-test-docker/results:/results:rw \
> docker.mgm.sipwise.com/janus-admin-stretch:I3370beedf30495497d01ec42eb292f90bce2b0a2 -c /code/t/testrunner
> ### Copying and moving files
> ################################################################################
> Finished main setup, now running tests ...
>
> > janus-admin@1.0.4 test /tmp/code
> > mocha -R spec --full-trace test/*-spec.js
>
> /usr/bin/env: 'node': No such file or directory
> npm ERR! Test failed. See above for more details.
> npm WARN This failure might be due to the use of legacy binary "node"
> npm WARN For further explanations, please read
> /usr/share/doc/nodejs/README.Debian
>
> npm ERR! not ok code 0
You can find more information about node/nodejs name convention here:
https://lists.debian.org/debian-devel-announce/2012/07/msg00002.html
So far, we need to install 'nodejs-legacy' to provide /usr/bin/node.
Change-Id: I3370beedf30495497d01ec42eb292f90bce2b0a2
Alexander Lutay
Markus Danek