Bug 35822 - Provide node package manager (npm) as package
Provide node package manager (npm) as package
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC (Generic)
UCS 4.0
Other Linux
: P5 normal (vote)
: UCS 4.0
Assigned To: Alexander Kramer
Alexander Kläser
: interim-2
Depends on:
Blocks: 35823
  Show dependency treegraph
 
Reported: 2014-09-04 11:58 CEST by Alexander Kläser
Modified: 2014-11-26 06:55 CET (History)
0 users

See Also:
What kind of report is it?: ---
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:


Attachments
diff before and after make install for npm (167.90 KB, text/plain)
2014-09-04 14:23 CEST, Alexander Kramer
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Kläser univentionstaff 2014-09-04 11:58:47 CEST
In order to correctly install certain dependencies that are needed for the build process of UMC, we need the node package manager to be available as package.
Comment 1 Alexander Kläser univentionstaff 2014-09-04 11:59:17 CEST
https://github.com/npm/npm
Comment 2 Alexander Kramer univentionstaff 2014-09-04 14:23:36 CEST
Created attachment 6091 [details]
diff before and after make install for npm
Comment 3 Alexander Kramer univentionstaff 2014-09-09 15:05:48 CEST
I removed the install target from the makefile, because we didn't need it anymore.
I added the copyright file from the stylus package into the npm package.

Still a little bit unsure about the control file, which I compared with the one of the stylus package:
- do we need: Depends ${shlibs:Depends}?
- is the Description okay for you? Took it from https://www.npmjs.org/doc/cli/npm.html

On my vm building and installing works fine.
Comment 4 Alexander Kramer univentionstaff 2014-09-09 15:38:19 CEST
r53487, r53489
Added copyright information to debian/rules
Comment 5 Alexander Kramer univentionstaff 2014-09-10 08:00:00 CEST
Failed build
Package: npm
Version: 0.0.0-1.4.201409091546
Branch: ucs_4.0-0

Cloning into 'git'...
error: Problem with the SSL CA cert (path? access rights?) while accessing https://github.com/npm/npm.git/info/refs
fatal: HTTP request failed
make[1]: *** [git] Error 128
make[1]: Leaving directory `/var/build/temp/tmp.4MJoUc6UFu/4.0-0-0-umc-dev/npm/npm-0.0.0'
dh_auto_build: make -j1 returned exit code 2
make: *** [build] Error 2
dpkg-buildpackage: error: debian/rules build gave error exit status 2
Comment 6 Alexander Kramer univentionstaff 2014-09-10 09:33:18 CEST
r53511: add ca-certificates build dependency
also adapted some copyrightinformation for debian/rules and Makefile.

Successful build
Package: npm
Version: 0.0.2-1.6.201409100925
User: akramer
Branch: ucs_4.0-0
Comment 7 Alexander Kramer univentionstaff 2014-09-10 14:11:18 CEST
I was able to install the package successfully from omar and also the stylus installation works fine.
Comment 8 Alexander Kläser univentionstaff 2014-09-11 14:49:36 CEST
Can you please adjust the debian/copyright file to reflect the correct NPM license (currently "stylus" is mentioned).
Comment 9 Alexander Kramer univentionstaff 2014-09-12 11:59:18 CEST
r53614
As requested i adapted the copyright information.
Took it from here:
https://github.com/npm/npm/blob/master/LICENSE

Successful build
Package: npm
Version: 0.0.2-2.7.201409121153
Branch: ucs_4.0-0
Comment 10 Alexander Kläser univentionstaff 2014-10-06 18:17:42 CEST
Looks good. I updated the package version to 1.4.27-1 to reflect the upstream version.

npm (1.4.27-1):
* Bug #35654: version bump to corresponding npm version
Comment 11 Stefan Gohmann univentionstaff 2014-11-26 06:55:22 CET
UCS 4.0-0 has been released:
 http://docs.univention.de/release-notes-4.0-0-en.html
 http://docs.univention.de/release-notes-4.0-0-de.html

If this error occurs again, please use "Clone This Bug".