Bug 961419

Summary: Review Request: anet - Ada Networking Library
Product: [Fedora] Fedora Reporter: Björn Persson <bjorn>
Component: Package ReviewAssignee: Christopher Meng <i>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: i, notting, package-review
Target Milestone: ---Flags: i: fedora-review+
gwync: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-07-30 14:01:15 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Björn Persson 2013-05-09 15:10:17 UTC
Spec URL: https://www.rombobjörn.se/packages/anet-0.2.2-1/anet.spec
SRPM URL: https://www.rombobjörn.se/packages/anet-0.2.2-1/anet-0.2.2-1.fc17.src.rpm
Description: Anet is a networking library for the Ada programming language. It supports, among other things, IPv6, Unix domain sockets, multicast, raw sockets, link layer sockets and Netlink.
Fedora Account System Username: rombobeorn

Comment 1 Christopher Meng 2013-05-09 15:40:26 UTC
IMO I think there is no need to write Summary(sv) and %description devel -l sv %{common_description_sv} same as well.

Some questions:

Why not naming it to libanet?

Why not using %doc in files section to handle the docs?

Comment 2 Björn Persson 2013-05-09 16:25:46 UTC
(In reply to comment #1)
> IMO I think there is no need to write Summary(sv) and %description devel -l
> sv %{common_description_sv} same as well.

The packaging guidelines allow it, and the review guidelines recommend it.

> Why not naming it to libanet?

The authors call it Anet almost everywhere. Except for the library file itself and the symlink, whose filenames must start with "lib" for technical reasons, only the tarball and the root directory inside it are called "libanet". Otherwise it's "anet" in filenames: /usr/include/anet, /usr/lib/anet, anet.gpr, anet.ads, anet.git et cetera. In prose it's generally "Anet", and "Ada Networking Library" in a few places.

> Why not using %doc in files section to handle the docs?

Because I want the HTML documentation to be bookmarkable. %doc generates a versioned directory name, so bookmarks would break on every update.

Comment 3 Christopher Meng 2013-07-26 09:46:49 UTC
Hi,

Since versioned docdir path will be replaced by unversioned, can you revise your spec?

https://fedoraproject.org/wiki/Changes/UnversionedDocdirs


Thanks.

Comment 4 Björn Persson 2013-07-28 17:01:44 UTC
Yes, now I can use %doc. (The documentation does get divided into two directories, anet and anet-devel, which is unnecessary but not a big problem.)

https://www.rombobjörn.se/packages/anet-0.2.2-2/anet.spec
https://www.rombobjörn.se/packages/anet-0.2.2-2/anet-0.2.2-2.fc17.src.rpm

Comment 5 Christopher Meng 2013-07-29 03:36:07 UTC
Only this:

anet.i686: I: enchant-dictionary-not-found sv

And, where is Ahven?

Will you package it?

If not I will package it ;)

Comment 6 Björn Persson 2013-07-29 11:21:12 UTC
(In reply to Christopher Meng from comment #5)
> Only this:
> 
> anet.i686: I: enchant-dictionary-not-found sv

So you don't have a Swedish spell checking library installed. Don't worry. I assure you that the Swedish spelling is correct. I get only bogus warnings when I run RPMlint.

> And, where is Ahven?

Its website is here:
http://ahven.stronglytyped.org/

> Will you package it?
> 
> If not I will package it ;)

I hope to get around to it eventually, but it's not my highest priority right now. If you have time then go ahead.

Ahven's GNAT project file builds only a static library, so you'll get to either convince Tero Koskinen to add an option to build a shared library, or convince the FPC that a static library is OK because it won't be linked into any installed programs, only testcases, or if both of those fail, carry a patch.

Comment 7 Christopher Meng 2013-07-29 23:38:09 UTC
Ok.

I think we can ignore it for a while.

APPROVED.

BTW if you want to install unversioned docs on F18/19, you can use:

%{!?_pkgdocdir: %global _pkgdocdir %{_docdir}/%{name}-%{version}}

Comment 8 Björn Persson 2013-07-30 11:13:53 UTC
Thanks for the review. I'll let F20 be the first release for Anet. At my current rate it will be a while before anything that will use it gets packaged.

New Package SCM Request
=======================
Package Name: anet
Short Description: Ada Networking Library
Owners: rombobeorn
Branches: 
InitialCC:

Comment 9 Gwyn Ciesla 2013-07-30 12:02:00 UTC
Git done (by process-git-requests).