LeoIannacone / npm2deb

tool to help debianize Node.js modules
GNU General Public License v3.0
46 stars 34 forks source link

add gitlab.com support #135

Closed guimard closed 5 years ago

guimard commented 5 years ago

may be incomplete. Fixes: #135

guimard commented 5 years ago

@pravi : works better now (b7fa621)

pravi commented 5 years ago

@shanavas786 @guimard

$ python3 npm2deb.py create @gitlab/eslint-config

Downloading source tarball file using debian/watch file...
uscan: Newest version of node-gitlab-eslint-config on remote site is 1.4.0, specified download version is 1.4.0
uscan error Could not read ..//gitlab-org/gitlab-eslint-config/-/archive/v1.4.0/gitlab-eslint-config-v1.4.0.tar.gz: No such file or directory at /usr/share/perl5/Devscripts/Output.pm line 61.

filenamemangle is not working. From gitlab-workhorse watch file, filenamemangle=s/.*v(.*)\/.*/gitlab-workhorse-$1.tar.gz/ works.

shanavas786 commented 5 years ago

@pravi Oh, I missed that error when I tried.

what makes me confused is http://gitlab.com/gitlab-org/gitlab-eslint-config/-/archive/v1.4.0/gitlab-eslint-config-v1.4.0.tar.gz is seems to be a valid url and browser downloads the file too.

Is this behavior is because of timeout @guimard mentioned earlier ?

pravi commented 5 years ago

Currently it is trying to create a file with '/' included in the name which fails because parent directories does not exist. The filenamemangle pattern shared above should fix it.

guimard commented 5 years ago

@pravi: should be fixed now

pravi commented 5 years ago

Now it is going back to npmregistry.

shanavas786 commented 5 years ago

It brings the uscan warn: unknown protocol for LWP: .*/archive/ warning back

guimard commented 5 years ago

@shanavas786, @pravi : fixed, missing space

shanavas786 commented 5 years ago

working fine. :+1: