https://bugzilla.redhat.com/show_bug.cgi?id=1572287
Bug ID: 1572287 Summary: Review Request: nodejs-istanbul-lib-hook - Hooks for require, vm and script used in istanbul Product: Fedora Version: rawhide Component: Package Review Assignee: nobody@fedoraproject.org Reporter: jsmith.fedora@gmail.com QA Contact: extras-qa@fedoraproject.org CC: package-review@lists.fedoraproject.org Blocks: 956806 (nodejs-reviews), 1269538 (IoT)
Spec URL: https://jsmith.fedorapeople.org/Packaging/nodejs-istanbul-lib-hook/nodejs-is... SRPM URL: https://jsmith.fedorapeople.org/Packaging/nodejs-istanbul-lib-hook/nodejs-is... Description: Hooks for require, vm and script used in istanbul Fedora Account System Username: jsmith
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=956806 [Bug 956806] Node.js Review Tracker https://bugzilla.redhat.com/show_bug.cgi?id=1269538 [Bug 1269538] Tracker for IoT on Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=1572287
Robert-André Mauchin zebob.m@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |zebob.m@gmail.com
--- Comment #1 from Robert-André Mauchin zebob.m@gmail.com --- - You seem to have forgotten to add the test dependencies:
Exécution_de(%check) : /bin/sh -e /var/tmp/rpm-tmp.E6jeb5 + umask 022 + cd /builddir/build/BUILD + cd package + /usr/lib/rpm/nodejs-symlink-deps /usr/lib/node_modules --check + /usr/bin/node -e 'require("./")' BUILDSTDERR: module.js:549 BUILDSTDERR: throw err; BUILDSTDERR: ^ BUILDSTDERR: Error: Cannot find module 'append-transform'
Exécution_de(%check) : /bin/sh -e /var/tmp/rpm-tmp.0TN5X6 + umask 022 + cd /builddir/build/BUILD + cd package + /usr/lib/rpm/nodejs-symlink-deps /usr/lib/node_modules --check + /usr/bin/node -e 'require("./")' + NODE_ENV=test + /usr/bin/mocha -R spec BUILDSTDERR: module.js:549 BUILDSTDERR: throw err; BUILDSTDERR: ^ BUILDSTDERR: Error: Cannot find module 'chai'
Add them:
%if 0%{?enable_tests} BuildRequires: mocha BuildRequires: npm(append-transform) BuildRequires: npm(chai) %endif
https://bugzilla.redhat.com/show_bug.cgi?id=1572287
Jared Smith jsmith.fedora@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED
--- Comment #2 from Jared Smith jsmith.fedora@gmail.com --- Could you please test again? It's possible I had an old version of the files uploaded to my web server.
https://bugzilla.redhat.com/show_bug.cgi?id=1572287
Robert-André Mauchin zebob.m@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |POST Assignee|nobody@fedoraproject.org |zebob.m@gmail.com Flags| |fedora-review+
--- Comment #3 from Robert-André Mauchin zebob.m@gmail.com --- - Latest version packaged - License ok - Builds in Mock - No rpmlint errors - Conforms to the Packaging Guidelines
Package approved.
https://bugzilla.redhat.com/show_bug.cgi?id=1572287
Jared Smith jsmith.fedora@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|POST |MODIFIED
--- Comment #4 from Jared Smith jsmith.fedora@gmail.com --- Repo requested at https://pagure.io/releng/fedora-scm-requests/issue/6414.
https://bugzilla.redhat.com/show_bug.cgi?id=1572287
--- Comment #5 from Jared Smith jsmith.fedora@gmail.com --- Repo requested at https://pagure.io/releng/fedora-scm-requests/issue/6416.
https://bugzilla.redhat.com/show_bug.cgi?id=1572287
--- Comment #6 from Gwyn Ciesla limburgher@gmail.com --- (fedrepo-req-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/nodejs-istanbul-lib-hook
https://bugzilla.redhat.com/show_bug.cgi?id=1572287
Peter Robinson pbrobinson@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks|1269538 (IoT) |
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=1269538 [Bug 1269538] Tracker for IoT on Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=1572287
Peter Robinson pbrobinson@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |1572709 (mozgw)
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=1572709 [Bug 1572709] Tracker for Mozilla IoT Gateway on Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=1572287
Jared Smith jsmith.fedora@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|MODIFIED |CLOSED Resolution|--- |RAWHIDE Last Closed| |2018-05-11 09:38:20
--- Comment #7 from Jared Smith jsmith.fedora@gmail.com --- Packaged in Rawhide, closing this issue
package-review@lists.fedoraproject.org