[Bug 1364699] Review Request: nodejs-linefix - Recursively repair line endings

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1364699

Jared Smith  changed:

   What|Removed |Added

 Status|MODIFIED|CLOSED
 Resolution|--- |RAWHIDE
Last Closed||2017-09-21 15:53:03



--- Comment #8 from Jared Smith  ---
In rawhide, closing bug

-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1364699] Review Request: nodejs-linefix - Recursively repair line endings

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1364699



--- Comment #7 from Gwyn Ciesla  ---
(fedrepo-req-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/nodejs-linefix

-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1364699] Review Request: nodejs-linefix - Recursively repair line endings

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1364699

Jared Smith  changed:

   What|Removed |Added

 Status|POST|MODIFIED



--- Comment #6 from Jared Smith  ---
'Requested package in ticket
https://pagure.io/releng/fedora-scm-requests/issue/1423'

-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1364699] Review Request: nodejs-linefix - Recursively repair line endings

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1364699

Robert-André Mauchin  changed:

   What|Removed |Added

 Status|ASSIGNED|POST
 CC||zebo...@gmail.com
   Assignee|vanmeeuwen+fedora@kolabsys. |zebo...@gmail.com
   |com |
  Flags|fedora-review?  |fedora-review+



--- Comment #5 from Robert-André Mauchin  ---
 - Get the license directly from upstream:

Source1:https://raw.githubusercontent.com/jhoff/linefix/master/LICENSE.txt

  And in %files

%license LICENSE.txt


Package otherwise accepted.

-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1364699] Review Request: nodejs-linefix - Recursively repair line endings

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1364699

Peter Robinson  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
-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1364699] Review Request: nodejs-linefix - Recursively repair line endings

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1364699

Peter Robinson  changed:

   What|Removed |Added

 Blocks||1387531 (Node-RED)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1387531
[Bug 1387531] Package tracker for supporting Node-RED
-- 
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1364699] Review Request: nodejs-linefix - Recursively repair line endings

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1364699



--- Comment #4 from Jared Smith  ---
Sorry, silly copy/paste error.

Spec URL:
https://jsmith.fedorapeople.org/Packaging/nodejs-linefix/nodejs-linefix.spec
SRPM URL:
https://jsmith.fedorapeople.org/Packaging/nodejs-linefix/nodejs-linefix-0.1.1-3.fc26.src.rpm

-- 
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
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1364699] Review Request: nodejs-linefix - Recursively repair line endings

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1364699



--- Comment #3 from Jeroen van Meeuwen  ---
You're linking /usr/bin/linefix to /usr/lib/node_modules/linefix/bin/cmd.js,
but that seems to need to be /usr/lib/node_modules/linefix/bin/fix.js

-- 
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
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1364699] Review Request: nodejs-linefix - Recursively repair line endings

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1364699

Jeroen van Meeuwen  changed:

   What|Removed |Added

  Flags||fedora-review?



--- Comment #1 from Jeroen van Meeuwen  ---
It's looking like bin/fix.js is supposed to be made available as
%{_bindir}/linefix (see README.md).

Please note that bin/fix.js has the wrong interpreter as well.

-- 
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
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1364699] Review Request: nodejs-linefix - Recursively repair line endings

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1364699

Jeroen van Meeuwen  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|nob...@fedoraproject.org|vanmeeuwen+fedora@kolabsys.
   ||com



-- 
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
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org