https://bugzilla.redhat.com/show_bug.cgi?id=1086217
--- Comment #18 from Fedora Update System ---
nodejs-strip-json-comments-1.0.2-1.el7 has been pushed to the Fedora EPEL 7
stable repository. If problems still persist, please make note of it in this
bug report.
--
You are receiving this mail
https://bugzilla.redhat.com/show_bug.cgi?id=1086217
--- Comment #17 from Fedora Update System ---
nodejs-strip-json-comments-1.0.2-1.el7 has been submitted as an update for
Fedora EPEL 7.
https://admin.fedoraproject.org/updates/nodejs-strip-json-comments-1.0.2-1.el7
--
You are receiving this
https://bugzilla.redhat.com/show_bug.cgi?id=1086217
--- Comment #16 from Jon Ciesla ---
Git done (by process-git-requests).
--
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=1086217
Jon Ciesla changed:
What|Removed |Added
Flags|fedora-cvs? |fedora-cvs+
--
You are receiving this m
https://bugzilla.redhat.com/show_bug.cgi?id=1086217
anish changed:
What|Removed |Added
Flags|needinfo?(apa...@redhat.com |fedora-cvs?
|) fedora-cvs+
https://bugzilla.redhat.com/show_bug.cgi?id=1086217
Parag changed:
What|Removed |Added
CC||apa...@redhat.com,
|
https://bugzilla.redhat.com/show_bug.cgi?id=1086217
Fedora Update System changed:
What|Removed |Added
Fixed In Version||nodejs-strip-json-comments-
https://bugzilla.redhat.com/show_bug.cgi?id=1086217
Jamie Nguyen changed:
What|Removed |Added
Status|ON_QA |CLOSED
CC|
https://bugzilla.redhat.com/show_bug.cgi?id=1086217
Fedora Update System changed:
What|Removed |Added
Status|MODIFIED|ON_QA
--- Comment #12 from Fed
https://bugzilla.redhat.com/show_bug.cgi?id=1086217
Fedora Update System changed:
What|Removed |Added
Status|ASSIGNED|MODIFIED
--
You are receiving
https://bugzilla.redhat.com/show_bug.cgi?id=1086217
--- Comment #11 from Fedora Update System ---
nodejs-strip-json-comments-0.1.3-4.fc20 has been submitted as an update for
Fedora 20.
https://admin.fedoraproject.org/updates/nodejs-strip-json-comments-0.1.3-4.fc20
--
You are receiving this ma
https://bugzilla.redhat.com/show_bug.cgi?id=1086217
Jon Ciesla changed:
What|Removed |Added
Flags|fedora-cvs? |fedora-cvs+
--
You are receiving this m
https://bugzilla.redhat.com/show_bug.cgi?id=1086217
--- Comment #10 from Jon Ciesla ---
Git done (by process-git-requests).
--
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=1086217
--- Comment #9 from anish ---
New Package SCM Request
===
Package Name: nodejs-strip-json-comments
Short Description: strip comments from json
Owners: anishpatil
Branches: f20
--
You are receiving this mail because:
You
https://bugzilla.redhat.com/show_bug.cgi?id=1086217
anish changed:
What|Removed |Added
Flags||fedora-cvs?
--
You are receiving this mail b
https://bugzilla.redhat.com/show_bug.cgi?id=1086217
Tom Hughes changed:
What|Removed |Added
Flags|fedora-review? |fedora-review+
--- Comment #8 from Tom H
https://bugzilla.redhat.com/show_bug.cgi?id=1086217
--- Comment #7 from anish ---
Hi Tom Hughes!
Thank you for your review comments, please find new SRPMS and spec file on
Spec URL: http://anishpatil.fedorapeople.org/nodejs-strip-json-comments.spec
SRPM URL:
http://anishpatil.fedorapeople
https://bugzilla.redhat.com/show_bug.cgi?id=1086217
--- Comment #6 from Tom Hughes ---
Upstream has released 0.1.3 now, which adds the license file. Once you've
updated to that I think this is good to go.
--
You are receiving this mail because:
You are on the CC list for the bug.
You are alwa
https://bugzilla.redhat.com/show_bug.cgi?id=1086217
--- Comment #5 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=1086217
--- Comment #4 from anish ---
Hi Tom Hughes!
Thank you for package review and review comments, I have incorporated few the
comments please find new spec and SRPM file on
Spec URL: http://anishpatil.fedorapeople.org/nodejs-strip-json-comment
https://bugzilla.redhat.com/show_bug.cgi?id=1086217
--- Comment #3 from Tom Hughes ---
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed
Issues
==
[!]: If the source package does not include license text(s)
https://bugzilla.redhat.com/show_bug.cgi?id=1086217
--- Comment #2 from anish ---
Hi Tom Hughes!
Thank you for package review and review comments, I have incorporated all the
comments please find new spec and SRPM file on
Spec URL: http://anishpatil.fedorapeople.org/nodejs-strip-json-comment
https://bugzilla.redhat.com/show_bug.cgi?id=1086217
--- Comment #1 from Tom Hughes ---
Preliminary comments:
* Needs an ExclusiveArch: %{nodejs_arches} noarch
* The second %setup is not needed
* You're installing cli.js twice, once with the cp then overwriting
it with the install
* Is the
https://bugzilla.redhat.com/show_bug.cgi?id=1086217
Tom Hughes changed:
What|Removed |Added
Status|NEW |ASSIGNED
CC|
https://bugzilla.redhat.com/show_bug.cgi?id=1086217
T.C. Hollingsworth changed:
What|Removed |Added
CC||tchollingswo...@gmail.com
25 matches
Mail list logo
