[Bug 214311] New: Review Request: kcometen3 - An OpenGL screensaver for KDE

[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/bugzilla/show_bug.cgi?id=214311

           Summary: Review Request: kcometen3 - An OpenGL screensaver for
                    KDE
           Product: Fedora Extras
           Version: devel
          Platform: All
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: normal
         Component: Package Review
        AssignedTo: nobody(a)fedoraproject.org
        ReportedBy: packages(a)amiga-hardware.com
         QAContact: fedora-package-review(a)redhat.com


Spec URL: http://dribble.org.uk/reviews/kcometen3.spec
SRPM URL: http://dribble.org.uk/reviews/kcometen3-1.1-3.src.rpm
Description: 

An OpenGL screensaver for KDE

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

[Bug 214311] Review Request: kcometen3 - An OpenGL screensaver for KDE

[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: kcometen3 - An OpenGL screensaver for KDE


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


cgoorah(a)yahoo.com.au changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
         AssignedTo|nobody(a)fedoraproject.org    |cgoorah(a)yahoo.com.au
OtherBugsDependingO|163776                      |163778
              nThis|                            |




------- Additional Comments From cgoorah(a)yahoo.com.au  2006-11-07 03:07 EST -------
I'll do the review :)

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

[Bug 214311] Review Request: kcometen3 - An OpenGL screensaver for KDE

[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: kcometen3 - An OpenGL screensaver for KDE


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





------- Additional Comments From cgoorah(a)yahoo.com.au  2006-11-07 03:27 EST -------
For the buildrequires:
-libjpeg-devel
-qt-devel >= 3.2
-/usr/bin/xmllint

These will be pulled down as dependencies for kdelibs-devel.
libxslt-devel depends on libxml2-devel

If not the package looks good, update this I'll do a full review of it.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

[Bug 214311] Review Request: kcometen3 - An OpenGL screensaver for KDE

[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: kcometen3 - An OpenGL screensaver for KDE


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





------- Additional Comments From packages(a)amiga-hardware.com  2006-11-07 05:39 EST
-------
If it's OK, can you continue with the review and I'll make those BR changes in 
addition to any changes needed by the review (if any) in the next release? It 
just saves having lots of minor changelog entries :-) Thanks.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

[Bug 214311] Review Request: kcometen3 - An OpenGL screensaver for KDE

[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: kcometen3 - An OpenGL screensaver for KDE


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





------- Additional Comments From cgoorah(a)yahoo.com.au  2006-11-07 08:28 EST -------
(In reply to comment #3)
>  If it's OK, can you continue with the review and I'll make
> those BR changes in 
>  addition to any changes needed by the review (if any) in the next release? It 
>  just saves having lots of minor changelog entries :-) Thanks. 
Ok, however do pinpoint to me these changes when you do :)

MUST Items:

- MUST: rpmlint's output is clean
- MUST: The package is named according to the Package Naming Guidelines.
- MUST: The spec file name matches the base package %{name}
- MUST: The package meets the Packaging Guidelines.
- MUST: The package is licensed (GPL) with an open-source compatible license and
meet other legal requirements as defined in the legal section of Packaging
Guidelines.
- MUST: The License field in the package spec file matches the actual license.
- MUST: the source package includes the text of the license(s) in its own file,
then that file, containing the text of the license(s) for the package is
included in %doc.
- MUST: The spec file must be written in American English.
- MUST: The spec file for the package is be legible. 
- MUST: The sources used to build the package must matches the upstream source,
as provided in the spec URL.
- MUST: The package successfully compiles and builds into binary rpms on at
least i386.
- MUST: All build dependencies is listed in BuildRequires.
- MUST: The spec file handles locales properly.
- MUST: If the package does not contain shared library files located in the
dynamic linker's default paths
- MUST: the package is not designed to be relocatable
- MUST: the package owns all directories that it creates.
- MUST: the package does not contain any duplicate files in the %files listing.
- MUST: Permissions on files are set properly.
- MUST: The package has a %clean section, which contains rm -rf %{buildroot} (or
$RPM_BUILD_ROOT).
- MUST: The package consistently uses macros, as described in the macros section
of Packaging Guidelines.
- MUST: The package contains code, or permissable content. This is described in
detail in the code vs. content section of Packaging Guidelines.
- MUST: There are no Large documentation files
- MUST: %doc does not affect the runtime of the application. To summarize: If it
is in %doc, the program must run properly if it is not present.
- MUST: There are no Header files or static libraries 
- MUST: The package does not contain library files with a suffix 
- MUST: Package does NOT contain any .la libtool archives
- MUST: Package containing GUI applications includes a %{name}.desktop file, and
that file must be properly installed with desktop-file-install in the %install
section.
- MUST: Package does not own files or directories already owned by other packages. 

SHOULD Items:

 - SHOULD: The source package does include license text(s) as COPYING
 - SHOULD: mock builds succcessfully in i386.
 - SHOULD: The reviewer tested that the package functions as described. A
package should not segfault instead of running, for example.
 - SHOULD: No scriptlets were used, those scriptlets must be sane. 
 - SHOULD: No subpackages present.

APPROVED

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

[Bug 214311] Review Request: kcometen3 - An OpenGL screensaver for KDE

[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: kcometen3 - An OpenGL screensaver for KDE


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


cgoorah(a)yahoo.com.au changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
OtherBugsDependingO|163778                      |163779
              nThis|                            |




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

[Bug 214311] Review Request: kcometen3 - An OpenGL screensaver for KDE

[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: kcometen3 - An OpenGL screensaver for KDE


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





------- Additional Comments From packages(a)amiga-hardware.com  2006-11-07 19:41 EST
-------
Thanks for the review. I'll remove the appropriate buildrequires and bump the
release field before import. I'll post a link to the update spec and srpm here
so you can check ;-)

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

[Bug 214311] Review Request: kcometen3 - An OpenGL screensaver for KDE

[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: kcometen3 - An OpenGL screensaver for KDE


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





------- Additional Comments From packages(a)amiga-hardware.com  2006-11-08 19:39 EST
-------
Updated version:

http://dribble.org.uk/reviews/kcometen3.spec
http://dribble.org.uk/reviews/kcometen3-1.1-4.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

[Bug 214311] Review Request: kcometen3 - An OpenGL screensaver for KDE

[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: kcometen3 - An OpenGL screensaver for KDE


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


packages(a)amiga-hardware.com changed:

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




------- Additional Comments From packages(a)amiga-hardware.com  2006-11-11 10:50 EST
-------
Built successfully. closing.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.