[Bug 960401] New: Review Request: xssstate - A simple tool to retrieve the X screensaver state

[Red Hat Bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=960401

Bug ID: 960401
           Summary: Review Request: xssstate - A simple tool to retrieve
                    the X screensaver state
           Product: Fedora
           Version: rawhide
         Component: Package Review
          Severity: medium
          Priority: medium
          Assignee: [e-mail]
          Reporter: [e-mail]
        QA Contact: [e-mail]
                CC: [e-mail],
                    [e-mail]
          Category: ---

Spec URL: http://psabata.fedorapeople.org/pkgs/xssstate/xssstate.spec
SRPM URL:
http://psabata.fedorapeople.org/pkgs/xssstate/xssstate-1.0-1.fc18.src.rpm
Description:
A simple tool to retrieve the X screensaver state.
Fedora Account System Username: psabata

[Bug 960401] Review Request: xssstate - A simple tool to retrieve the X screensaver state

[Red Hat Bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=960401

Petr Pisar  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]
           Assignee|[e-mail]    |[e-mail]
              Flags|                            |fedora-review?

[Bug 960401] Review Request: xssstate - A simple tool to retrieve the X screensaver state

[Red Hat Bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=960401

Petr Pisar  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED

--- Comment #1 from Petr Pisar  ---
URL and Source0 are usable. Ok.
Source tar ball is original (SHA-256:
e5b60a6e48133944ef6f3c2e0b7bd40e0d84d2d97ed9790c85c382bc1998bf04). Ok.
License is Ok.
Summary and description is good.

TODO: There are two bugs in the source:

printf("%ld\n", info->til_or_since);
 printf("%ld\n", info->idle);

Both variables are of type unsigned long, but the formatting strings are %ld.
See <X11/extensions/scrnsaver.h> for XScreenSaverInfo type definitions.

$ rpmlint xssstate.spec ../SRPMS/xssstate-1.0-1.fc20.src.rpm
../RPMS/x86_64/xssstate-*
xssstate.src: W: spelling-error Summary(en_US) screensaver -> screen saver,
screen-saver, screens aver
xssstate.src: W: spelling-error %description -l en_US screensaver -> screen
saver, screen-saver, screens aver
xssstate.x86_64: W: spelling-error Summary(en_US) screensaver -> screen saver,
screen-saver, screens aver
xssstate.x86_64: W: spelling-error %description -l en_US screensaver -> screen
saver, screen-saver, screens aver
3 packages and 1 specfiles checked; 0 errors, 4 warnings.

TODO: Consider spelling `screensaver' with a hyphen or a space.

$ rpm -q -lv -p  ../RPMS/x86_64/xssstate-1.0-1.fc20.x86_64.rpm
-rwxr-xr-x    1 root    root                    11368 May  7 12:49
/usr/bin/xssstate
drwxr-xr-x    2 root    root                        0 May  7 12:49
/usr/share/doc/xssstate-1.0
-rw-r--r--    1 root    root                     1138 Feb  1 13:10
/usr/share/doc/xssstate-1.0/LICENSE
-rw-r--r--    1 root    root                      784 Feb  1 13:10
/usr/share/doc/xssstate-1.0/README
-rwxr-xr-x    1 root    root                      360 Feb  1 13:10
/usr/share/doc/xssstate-1.0/xsidle.sh
-rw-r--r--    1 root    root                      443 May  7 12:49
/usr/share/man/man1/xssstate.1.gz
File layout and permissions are Ok.

$ rpm -q --requires -p  ../RPMS/x86_64/xssstate-1.0-1.fc20.x86_64.rpm | sort 
/bin/sh
libc.so.6()(64bit)
libc.so.6(GLIBC_2.2.5)(64bit)
libc.so.6(GLIBC_2.3.4)(64bit)
libX11.so.6()(64bit)
libXss.so.1()(64bit)
rpmlib(CompressedFileNames) <= 3.0.4-1
rpmlib(FileDigests) <= 4.6.0-1
rpmlib(PayloadFilesHavePrefix) <= 4.0-1
rpmlib(PayloadIsXz) <= 5.2-1
rtld(GNU_HASH)

FIX: Do not scan documentation for dependencies (/bin/sh).

$ rpm -q --provides -p ../RPMS/x86_64/xssstate-1.0-1.fc20.x86_64.rpm | sort 
xssstate = 1.0-1.fc20
xssstate(x86-64) = 1.0-1.fc20
Binary provides are Ok.

$ resolvedeps rawhide ../RPMS/x86_64/xssstate-1.0-1.fc20.x86_64.rpm 
Binary dependencies resolvable. Ok.

Package builds in F20
(http://koji.fedoraproject.org/koji/taskinfo?taskID=5338920). Ok.

Otherwise the package is line with Fedora packaging guidelines.

Please correct all `FIX' issues, consider fixing `TODO' items, and provide new
spec file.

Resolution: Package NOT approved.

[Bug 960401] Review Request: xssstate - A simple tool to retrieve the X screensaver state

[Red Hat Bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=960401

--- Comment #2 from Petr Šabata  ---
(In reply to comment #1)
> URL and Source0 are usable. Ok.
> Source tar ball is original (SHA-256:
> e5b60a6e48133944ef6f3c2e0b7bd40e0d84d2d97ed9790c85c382bc1998bf04). Ok.
> License is Ok.
> Summary and description is good.
>
> TODO: There are two bugs in the source:
>
> printf("%ld\n", info->til_or_since);
>  printf("%ld\n", info->idle);
>
> Both variables are of type unsigned long, but the formatting strings are
> %ld. See <X11/extensions/scrnsaver.h> for XScreenSaverInfo type definitions.

Patched to %lu and submitted upstream.
> $ rpmlint xssstate.spec ../SRPMS/xssstate-1.0-1.fc20.src.rpm
> ../RPMS/x86_64/xssstate-*
> xssstate.src: W: spelling-error Summary(en_US) screensaver -> screen saver,
> screen-saver, screens aver
> xssstate.src: W: spelling-error %description -l en_US screensaver -> screen
> saver, screen-saver, screens aver
> xssstate.x86_64: W: spelling-error Summary(en_US) screensaver -> screen
> saver, screen-saver, screens aver
> xssstate.x86_64: W: spelling-error %description -l en_US screensaver ->
> screen saver, screen-saver, screens aver
> 3 packages and 1 specfiles checked; 0 errors, 4 warnings.
>
> TODO: Consider spelling `screensaver' with a hyphen or a space.

Switched to "screen saver" to be in line with other packages (e.g.
xscreensaver).
> $ rpm -q -lv -p  ../RPMS/x86_64/xssstate-1.0-1.fc20.x86_64.rpm
> -rwxr-xr-x    1 root    root                    11368 May  7 12:49
> /usr/bin/xssstate
> drwxr-xr-x    2 root    root                        0 May  7 12:49
> /usr/share/doc/xssstate-1.0
> -rw-r--r--    1 root    root                     1138 Feb  1 13:10
> /usr/share/doc/xssstate-1.0/LICENSE
> -rw-r--r--    1 root    root                      784 Feb  1 13:10
> /usr/share/doc/xssstate-1.0/README
> -rwxr-xr-x    1 root    root                      360 Feb  1 13:10
> /usr/share/doc/xssstate-1.0/xsidle.sh
> -rw-r--r--    1 root    root                      443 May  7 12:49
> /usr/share/man/man1/xssstate.1.gz
> File layout and permissions are Ok.
>
> $ rpm -q --requires -p  ../RPMS/x86_64/xssstate-1.0-1.fc20.x86_64.rpm | sort 
> /bin/sh
> libc.so.6()(64bit)
> libc.so.6(GLIBC_2.2.5)(64bit)
> libc.so.6(GLIBC_2.3.4)(64bit)
> libX11.so.6()(64bit)
> libXss.so.1()(64bit)
> rpmlib(CompressedFileNames) <= 3.0.4-1
> rpmlib(FileDigests) <= 4.6.0-1
> rpmlib(PayloadFilesHavePrefix) <= 4.0-1
> rpmlib(PayloadIsXz) <= 5.2-1
> rtld(GNU_HASH)
>
> FIX: Do not scan documentation for dependencies (/bin/sh).

Done.
> $ rpm -q --provides -p ../RPMS/x86_64/xssstate-1.0-1.fc20.x86_64.rpm | sort 
> xssstate = 1.0-1.fc20
> xssstate(x86-64) = 1.0-1.fc20
> Binary provides are Ok.
>
> $ resolvedeps rawhide ../RPMS/x86_64/xssstate-1.0-1.fc20.x86_64.rpm 
> Binary dependencies resolvable. Ok.
>
> Package builds in F20
> (http://koji.fedoraproject.org/koji/taskinfo?taskID=5338920). Ok.
>
> Otherwise the package is line with Fedora packaging guidelines.
>
> Please correct all `FIX' issues, consider fixing `TODO' items, and provide
> new spec file.
>
> Resolution: Package NOT approved.

The original links now point to the updated package.

[Bug 960401] Review Request: xssstate - A simple tool to retrieve the X screensaver state

[Red Hat Bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=960401

Petr Pisar  changed:

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

--- Comment #3 from Petr Pisar  ---
Spec file changes:

--- xssstate.spec.old   2013-05-07 09:30:04.000000000 +0200
+++ xssstate.spec       2013-05-07 17:03:43.373000000 +0200
@@ -1,10 +1,11 @@
 Name:          xssstate
 Version:       1.0
 Release:       1%{?dist}
-Summary:       A simple tool to retrieve the X screensaver state
+Summary:       A simple tool to retrieve the X screen saver state
 License:       MIT
 URL:           http://tools.suckless.org/%%7Bname%7D
 Source0:       http://dl.suckless.org/tools/%%7Bname%7D-%%7Bversion%7D.tar.gz
+Patch0:        %{name}-1.0-format.patch
 BuildRequires: libXScrnSaver-devel

%description
@@ -12,11 +13,13 @@

%prep
 %setup -q
+%patch0 -p1
 sed -e 's|PREFIX = /usr/local|PREFIX = %{_prefix}|' \
     -e 's|LIBS = -L/usr/lib -lc -lX11 -lXss|LIBS = -L%{_libdir} -lc -lX11
-lXss|' \
     -e 's|CFLAGS = -g -std=c99 -pedantic -Wall -O0 ${INCS} ${CPPFLAGS}|CFLAGS
= %{optflags} ${INCS} ${CPPFLAGS}|' \
    -i config.mk
 sed -i 's|^\t@|\t|' Makefile
+chmod -x xsidle.sh

%build
 make %{?_smp_mflags}
> TODO: There are two bugs in the source:

+Patch0:        %{name}-1.0-format.patch
And I saw that the patch was good. Ok.
> TODO: Consider spelling `screensaver' with a hyphen or a space.

-Summary:       A simple tool to retrieve the X screensaver state
+Summary:       A simple tool to retrieve the X screen saver state
Ok.
> FIX: Do not scan documentation for dependencies (/bin/sh).

+chmod -x xsidle.sh
Ok.

$ rpmlint xssstate.spec ../SRPMS/xssstate-1.0-1.fc20.src.rpm
../RPMS/x86_64/xssstate-*
3 packages and 1 specfiles checked; 0 errors, 0 warnings.
rpmlint is Ok.

$ rpm -q -lv -p  ../RPMS/x86_64/xssstate-1.0-1.fc20.x86_64.rpm
-rwxr-xr-x    1 root    root                    11368 May  7 17:08
/usr/bin/xssstate
drwxr-xr-x    2 root    root                        0 May  7 17:08
/usr/share/doc/xssstate-1.0
-rw-r--r--    1 root    root                     1138 Feb  1 13:10
/usr/share/doc/xssstate-1.0/LICENSE
-rw-r--r--    1 root    root                      784 Feb  1 13:10
/usr/share/doc/xssstate-1.0/README
-rw-r--r--    1 root    root                      360 Feb  1 13:10
/usr/share/doc/xssstate-1.0/xsidle.sh
-rw-r--r--    1 root    root                      443 May  7 17:08
/usr/share/man/man1/xssstate.1.gz
File permissions and layout are Ok.

$ rpm -q --requires -p ../RPMS/x86_64/xssstate-1.0-1.fc20.x86_64.rpm | sort
libc.so.6()(64bit)
libc.so.6(GLIBC_2.2.5)(64bit)
libc.so.6(GLIBC_2.3.4)(64bit)
libX11.so.6()(64bit)
libXss.so.1()(64bit)
rpmlib(CompressedFileNames) <= 3.0.4-1
rpmlib(FileDigests) <= 4.6.0-1
rpmlib(PayloadFilesHavePrefix) <= 4.0-1
rpmlib(PayloadIsXz) <= 5.2-1
rtld(GNU_HASH)
Binary requires are Ok.

Package builds in F20
(http://koji.fedoraproject.org/koji/taskinfo?taskID=5340105). Ok.

Package is good.

Resolution: Package APPROVED.

[Bug 960401] Review Request: xssstate - A simple tool to retrieve the X screensaver state

[Red Hat Bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=960401

--- Comment #4 from Petr Šabata  ---
Thank you.
FYI, patch is upstream now.

[Bug 960401] Review Request: xssstate - A simple tool to retrieve the X screen saver state

[Red Hat Bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=960401

Petr Šabata  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|Review Request: xssstate -  |Review Request: xssstate -
                   |A simple tool to retrieve   |A simple tool to retrieve
                   |the X screensaver state     |the X screen saver state
              Flags|                            |fedora-cvs?

--- Comment #5 from Petr Šabata  ---
New Package SCM Request
=======================
Package Name: xssstate
Short Description: A simple tool to retrieve the X screen saver state
Owners: psabata
Branches: f17 f18 f19
InitialCC:

[Bug 960401] Review Request: xssstate - A simple tool to retrieve the X screen saver state

[Red Hat Bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=960401

Jon Ciesla  changed:

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

[Bug 960401] Review Request: xssstate - A simple tool to retrieve the X screen saver state

[Red Hat Bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=960401

--- Comment #6 from Jon Ciesla  ---
Git done (by process-git-requests).

[Bug 960401] Review Request: xssstate - A simple tool to retrieve the X screen saver state

[Red Hat Bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=960401

Fedora Update System  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |MODIFIED

[Bug 960401] Review Request: xssstate - A simple tool to retrieve the X screen saver state

[Red Hat Bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=960401

--- Comment #7 from Fedora Update System  ---
xssstate-1.1-1.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/xssstate-1.1-1.fc19

[Bug 960401] Review Request: xssstate - A simple tool to retrieve the X screen saver state

[Red Hat Bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=960401

--- Comment #8 from Fedora Update System  ---
xssstate-1.1-1.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/xssstate-1.1-1.fc18

[Bug 960401] Review Request: xssstate - A simple tool to retrieve the X screen saver state

[Red Hat Bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=960401

--- Comment #9 from Fedora Update System  ---
xssstate-1.1-1.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/xssstate-1.1-1.fc17

[Bug 960401] Review Request: xssstate - A simple tool to retrieve the X screen saver state

[Red Hat Bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=960401

--- Comment #10 from Petr Šabata  ---
Thank you, Jon.

[Bug 960401] Review Request: xssstate - A simple tool to retrieve the X screen saver state

[Red Hat Bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=960401

--- Comment #11 from Fedora Update System  ---
xssstate-1.1-1.fc19 has been pushed to the Fedora 19 stable repository.

[Bug 960401] Review Request: xssstate - A simple tool to retrieve the X screen saver state

[Red Hat Bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=960401

--- Comment #12 from Fedora Update System  ---
xssstate-1.1-1.fc18 has been pushed to the Fedora 18 stable repository.

[Bug 960401] Review Request: xssstate - A simple tool to retrieve the X screen saver state

[Red Hat Bugzilla]

Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=960401

--- Comment #13 from Fedora Update System  ---
xssstate-1.1-1.fc17 has been pushed to the Fedora 17 stable repository.

[Bug 960401] Review Request: xssstate - A simple tool to retrieve the X screen saver state

[Red Hat Bugzilla]

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

Petr Šabata  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|MODIFIED                    |CLOSED
   Fixed In Version|                            |xssstate-1.1-1.fc17
         Resolution|---                         |CURRENTRELEASE
        Last Closed|                            |2014-04-03 08:42:15