[Bug 790525] Review Request: rubygem-dynect_rest - Dynect REST API library

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

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

John (J5) Palmieri <johnp@xxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |johnp@xxxxxxxxxx

--- Comment #3 from John (J5) Palmieri <johnp@xxxxxxxxxx> 2012-02-14 18:06:32 EST ---
> W: spelling-error Summary(en_US) Dynect -> Connect
> Its the correct name of the service to which this gem communicates

We can just wave that as a false positive

> Various unexpanded-macro warnings against file names in the gems doc dirs
> I take it this is commonly seen with ruby gems.  If there is action I need to
> take to resolve it please let me know.

Not sure, I'll take a look at it

> W: wrong-file-end-of-line-encoding
> /usr/share/gems/doc/dynect_rest-0.4.0/ri/cache.ri
> Its a binary file so I'm not quite sure what is causing rpmlint to issue this
> warning.

Binaries really shouldn't be in the doc directory. Do other packages have these
cache files?  Rpmlint is just making sure things should be in their place and
is pretty pedantic. Warnings are usually just things it thinks you should
check.  If you can justify it then we can wave it.  I'll run this through
review tomorrow.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
_______________________________________________
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]