[Bug 823446] Review Request: libradius - This is a library to generate RADIUS authentication request

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

 



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

Simone Caronni <negativo17@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|needinfo?                   |

--- Comment #8 from Simone Caronni <negativo17@xxxxxxxxx> ---
(In reply to comment #2)
> tl;dr version:
> The libradius(3) belongs in the -devel package. It's not useful for runtime.
> The version is wrong, doesn't match upstream and doesn't match the %changelog
>
> Not a blocker to packaging:
> It would be better to avoid the dep on libmd if possible. There are many
> other libraries in Fedora already that can do MD5 hashes. Consider porting
> the dependency to the 'nss' package for crypto certification. See
> http://crypto.stanford.edu/firefox-rhash/ for a good starting point.

Updated package with the fixes:

Spec URL: http://slaanesh.fedorapeople.org/libradius.spec
SRPM URL: http://slaanesh.fedorapeople.org/libradius-20040827-2.fc17.src.rpm

- Fixed changelog version that was causing rpmlint errors.
- Moved libradius(3) manpage in -devel package.

I'm not able to port the MD5 calls to the nss library; if you can supply the
patch to remove the dependency that would be great. As it's not a blocker to
packaging that can also be set later on by setting an Obsolete flag in the spec
file wrt the libmd library as this is the only package requiring it.

Thanks,
--Simone

-- 
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]