https://bugzilla.redhat.com/show_bug.cgi?id=1285412
Piotr Popieluch changed:
What|Removed |Added
Blocks||1296302
Referenced Bugs:
https://
https://bugzilla.redhat.com/show_bug.cgi?id=1285412
--- Comment #5 from Piotr Popieluch ---
Ok, to keep thing going I've updated the spec.
Tom, are you ok to review my changes?
Spec URL: https://piotrp.fedorapeople.org/nodejs-ebnf-parser.spec
SRPM URL:
https://piotrp.fedorapeople.org/nodejs-
https://bugzilla.redhat.com/show_bug.cgi?id=1285412
--- Comment #6 from Tom Hughes ---
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed
= MUST items =
Generic:
[x]: Package is licensed with an open-so
https://bugzilla.redhat.com/show_bug.cgi?id=1285412
--- Comment #7 from Tom Hughes ---
Yes that looks fine. Jared, are you happy to use that spec?
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
__
https://bugzilla.redhat.com/show_bug.cgi?id=1285412
--- Comment #8 from Jared Smith ---
Yes, I just returned from vacation, and I'd be happy to use the updated spec
provided by Piotr during my absence.
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always
https://bugzilla.redhat.com/show_bug.cgi?id=1285412
Tom Hughes changed:
What|Removed |Added
Flags|fedora-review? |fedora-review+
--- Comment #9 from Tom H
https://bugzilla.redhat.com/show_bug.cgi?id=1285412
--- Comment #10 from Jon Ciesla ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/nodejs-ebnf-parser
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about
https://bugzilla.redhat.com/show_bug.cgi?id=1285412
Jared Smith changed:
What|Removed |Added
Status|ASSIGNED|CLOSED
Resolution|---
https://bugzilla.redhat.com/show_bug.cgi?id=1285412
Jared Smith changed:
What|Removed |Added
Summary|Review Request: nodejs-ebnf |Review Request:
|- A pa
https://bugzilla.redhat.com/show_bug.cgi?id=1285412
Tom Hughes changed:
What|Removed |Added
Status|NEW |ASSIGNED
CC|
https://bugzilla.redhat.com/show_bug.cgi?id=1285412
--- Comment #1 from Tom Hughes ---
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed
= MUST items =
Generic:
[x]: Package is licensed with an open-so
https://bugzilla.redhat.com/show_bug.cgi?id=1285412
--- Comment #2 from Tom Hughes ---
Needs BRs on npm(lex-parser) and npm(test) when tests are enables.
Also it does have a run time dependency on lex-parser that is not declared in
the package.json because ebnf-parser.js requires it. It will w
https://bugzilla.redhat.com/show_bug.cgi?id=1285412
Piotr Popieluch changed:
What|Removed |Added
CC||piotr1...@gmail.com
--- Comment #3
https://bugzilla.redhat.com/show_bug.cgi?id=1285412
--- Comment #4 from Piotr Popieluch ---
npm-test is now in rawhide and has buildroot overrides. Tests can be enabled.
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this p
14 matches
Mail list logo
