[Bug 1086245] Review Request: nodejs-jsonstream -streaming JSON.parse and stringify for Node.js

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



--- Comment #7 from T.C. Hollingsworth <tchollingsworth@xxxxxxxxx> ---
As I said before, personally I would capitalize the RPM package name too so
that users who are aware that there were once two jsonstreams know for sure
which one it is. The guidelines say "You should generally use lowercase and
turn underscores into dashes unless there's a compelling reason to follow a
different upstream convention."  I think the fact that there once were two
packages with the same name differently cased is a compelling enough reason to
respect the upstream casing in this particular instance. But with the conflict
resolved upstream I don't feel too strongly about it anymore so ultimately it's
up to you two. :-)

But yes, regardless of what the rpm is named the capitalization needs to stay
in the filesystem, since as you point out require() is case-sensitive when used
on case-sensitive filesystems.

-- 
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@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review





[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]