[Bug 434890] New: Review Request: LinLog - A ham radio logbook for linux

[Red Hat Bugzilla]

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




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

           Summary: Review Request: LinLog - A ham radio logbook for linux
           Product: Fedora
           Version: rawhide
          Platform: All
        OS/Version: Linux
            Status: NEW
          Severity: medium
          Priority: medium
         Component: Package Review
        AssignedTo: nobody(a)fedoraproject.org
        ReportedBy: bob(a)fedoraunity.org
         QAContact: extras-qa(a)fedoraproject.org
                CC: [e-mail],notting(a)redhat.com


Spec URL: http://bjensen.fedorapeople.org/pkgs/hams/SPECS/LinLog.spec
SRPM URL: http://bjensen.fedorapeople.org/pkgs/hams/SRPMS/LinLog-0.3-3.fc8.src.rpm
Description: LinLogBook is a highly configurable amateur radio logbook for linux.

It uses an sql- database to store its data. For the ease of use sqlite 3 is 
used but it should be possible to use other databases like mysql, for instance.

In testing package builds clean on Fedora 7 and Fedora 8, all resulting rpms are rpmlint
clean. This is my first package that I had to include a .desktop file and icon for. Please
use extra care when looking at those parts to make sure I did in fact get it right.

-- 
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, or are watching someone who is.

[Bug 434890] Review Request: LinLog - A ham radio logbook for linux

[Red Hat Bugzilla]

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

Summary: Review Request: LinLog - A ham radio logbook for linux


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


kanarip(a)kanarip.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|nobody(a)fedoraproject.org    |kanarip(a)kanarip.com
             Status|NEW                         |ASSIGNED
               Flag|                            |fedora-review?




-- 
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, or are watching someone who is.

[Bug 434890] Review Request: LinLog - A ham radio logbook for linux

[Red Hat Bugzilla]

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

Summary: Review Request: LinLog - A ham radio logbook for linux


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





------- Additional Comments From kanarip(a)kanarip.com  2008-02-26 05:14 EST -------
RPM Lint: OK
Package name: OK
Spec file: OK
License: OK, GPLv2+
Actual License: OK, but source (src/linlog.cpp) says "GPL see COPYING file
shipped", COPYING file just says GPLv2
%doc License: OK
Spec file language: OK
Spec file readable: OK
Upstream source vs. used tarball: OK (md5sum match)
Compile and Build:
 - F-7: OK
 - F-8: OK
 - rawhide: OK
 - EL-5: FAILED: No package qt4-devel, but not a target, so it's OK.

Applicable Package Guidelines:
 - -debuginfo package: OK (~1.9M)

Locales: N/A
Shared libs: N/A
Relocatable: N/A
Directory and file ownership: OK
No duplicate files in %files: OK
File Permissions: OK
Macro usage: OK
Code vs. Content: OK
(Large) Documentation: .pdf file (manual) included, not very large though: OK
%doc affecting runtime: OK
Header files in -devel package: N/A
Static Libraries in -static package: N/A
pkgconfig Requires: N/A
Library files: N/A
Devel requires base package: N/A
.la libtool archives: N/A
Duplicate ownership of files/directories: OK
Remove BuildRoot: OK
UTF-8 filenames: OK

Problems:
The license of the package should be GPLv2 according to src/linlog.cpp, and I
can't find any other mention off 'GPL' or 'or later' in the code using
a simple
'grep -rin <string> *'. Please let me know where it says v2+ if it is really
v2+.

Another thing: You did well on the .desktop and icon file ;-)

-- 
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, or are watching someone who is.

[Bug 434890] Review Request: LinLog - A ham radio logbook for linux

[Red Hat Bugzilla]

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

Summary: Review Request: LinLog - A ham radio logbook for linux


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





------- Additional Comments From bob(a)fedoraunity.org  2008-02-26 08:47 EST -------
Corrected License.

Spec URL: http://bjensen.fedorapeople.org/pkgs/hams/SPECS/LinLog.spec
SRPM URL: http://bjensen.fedorapeople.org/pkgs/hams/SRPMS/LinLog-0.3-4.fc8.src.rpm

-- 
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, or are watching someone who is.

[Bug 434890] Review Request: LinLog - A ham radio logbook for linux

[Red Hat Bugzilla]

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

Summary: Review Request: LinLog - A ham radio logbook for linux


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


kanarip(a)kanarip.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-review?              |fedora-review+




------- Additional Comments From kanarip(a)kanarip.com  2008-02-26 09:03 EST -------
APPROVED

-- 
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, or are watching someone who is.

[Bug 434890] Review Request: LinLog - A ham radio logbook for linux

[Red Hat Bugzilla]

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

Summary: Review Request: LinLog - A ham radio logbook for linux


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





------- Additional Comments From bob(a)fedoraunity.org  2008-02-26 09:26 EST -------
Upon further investigation this should in fact be GPLv2+

-- 
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, or are watching someone who is.

[Bug 434890] Review Request: LinLog - A ham radio logbook for linux

[Red Hat Bugzilla]

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

Summary: Review Request: LinLog - A ham radio logbook for linux


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





------- Additional Comments From kanarip(a)kanarip.com  2008-02-26 09:59 EST -------
Correct, it's in multiple source files. Sorry, my bad.

-- 
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, or are watching someone who is.

[Bug 434890] Review Request: LinLog - A ham radio logbook for linux

[Red Hat Bugzilla]

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

Summary: Review Request: LinLog - A ham radio logbook for linux


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





------- Additional Comments From kanarip(a)kanarip.com  2008-02-28 16:38 EST -------
Ping, pending a CVS request Bob... ;-)

-- 
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, or are watching someone who is.

[Bug 434890] Review Request: LinLog - A ham radio logbook for linux

[Red Hat Bugzilla]

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

Summary: Review Request: LinLog - A ham radio logbook for linux


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


bob(a)fedoraunity.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|                            |fedora-cvs?




------- Additional Comments From bob(a)fedoraunity.org  2008-02-28 23:46 EST -------
New Package CVS Request
=======================
Package Name: LinLog
Short Description: A ham radio logbook for Linux
Owners: bjensen,sconklin
Branches: F-7 F-8
InitialCC: sindrepb
Cvsextras Commits: Yes

-- 
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, or are watching someone who is.

[Bug 434890] Review Request: LinLog - A ham radio logbook for linux

[Red Hat Bugzilla]

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

Summary: Review Request: LinLog - A ham radio logbook for linux


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


kevin(a)tummy.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-cvs?                 |fedora-cvs+




------- Additional Comments From kevin(a)tummy.com  2008-02-29 02:33 EST -------
cvs done.

-- 
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, or are watching someone who is.

[Bug 434890] Review Request: LinLog - A ham radio logbook for linux

[Red Hat Bugzilla]

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

Summary: Review Request: LinLog - A ham radio logbook for linux


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





------- Additional Comments From updates(a)fedoraproject.org  2008-02-29 10:25 EST -------
LinLog-0.3-4.fc8 has been submitted as an update for Fedora 8

-- 
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, or are watching someone who is.

[Bug 434890] Review Request: LinLog - A ham radio logbook for linux

[Red Hat Bugzilla]

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

Summary: Review Request: LinLog - A ham radio logbook for linux


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





------- Additional Comments From updates(a)fedoraproject.org  2008-02-29 10:27 EST -------
LinLog-0.3-4.fc7 has been submitted as an update for Fedora 7

-- 
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, or are watching someone who is.

[Bug 434890] Review Request: LinLog - A ham radio logbook for linux

[Red Hat Bugzilla]

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

Summary: Review Request: LinLog - A ham radio logbook for linux


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


bob(a)fedoraunity.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |CLOSED
         Resolution|                            |NEXTRELEASE




------- Additional Comments From bob(a)fedoraunity.org  2008-02-29 10:49 EST -------
Thank You Kanarip for the review.
Thank You nirik for handling cvs.

-- 
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, or are watching someone who is.

[Bug 434890] Review Request: LinLog - A ham radio logbook for linux

[Red Hat Bugzilla]

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

Summary: Review Request: LinLog - A ham radio logbook for linux


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





------- Additional Comments From updates(a)fedoraproject.org  2008-03-01 04:27 EST -------
LinLog-0.3-4.fc7 has been pushed to the Fedora 7 stable repository.  If problems still
persist, please make note of it in this bug report.

-- 
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, or are watching someone who is.

[Bug 434890] Review Request: LinLog - A ham radio logbook for linux

[Red Hat Bugzilla]

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

Summary: Review Request: LinLog - A ham radio logbook for linux


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





------- Additional Comments From updates(a)fedoraproject.org  2008-03-01 04:27 EST -------
LinLog-0.3-4.fc8 has been pushed to the Fedora 8 stable repository.  If problems still
persist, please make note of it in this bug report.

-- 
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, or are watching someone who is.