Bug 1462465 (cmatrix) - Review Request: cmatrix - A scrolling 'Matrix'-like screen
Summary: Review Request: cmatrix - A scrolling 'Matrix'-like screen
Keywords:
Status: CLOSED RAWHIDE
Alias: cmatrix
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Zbigniew Jędrzejewski-Szmek
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
: 584111 (view as bug list)
Depends On:
Blocks: hollywood
TreeView+ depends on / blocked
 
Reported: 2017-06-17 21:18 UTC by Raphael Groner
Modified: 2017-08-13 00:12 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2017-08-13 00:12:29 UTC
Type: ---
Embargoed:
zbyszek: fedora-review+

Attachments (Terms of Use)

Description Raphael Groner 2017-06-17 21:18:16 UTC 
Spec URL: http://raphgro.fedorapeople.org//cmatrix.spec
SRPM URL: http://raphgro.fedorapeople.org//cmatrix-1.2a-1.fc26.src.rpm

Description:
Let's see the cool scrolling lines from the famous movie 'The Matrix'.
Comment 1 Raphael Groner 2017-06-17 21:20:21 UTC 
*** Bug 584111 has been marked as a duplicate of this bug. ***
Comment 2 Raphael Groner 2017-06-17 21:37:08 UTC 
Task info: https://koji.fedoraproject.org/koji/taskinfo?taskID=20057122
Comment 3 Zbigniew Jędrzejewski-Szmek 2017-07-15 03:17:57 UTC 
> Suggests:            console-setup
Doesn't this duplicate functionality provided by kbd and systemd-vconsole-setup? Are you sure it's still useful?

> Suggests:            aterm
> Suggests:            xterm
Both of those are very old, and work badly on hidpi screens. Are they better than the default terminal emulators?

Braces around the %post and %postun scriptlets are unneeded.

+ package name is OK
+ license is acceptable (GPLv2+)
+ license is specified correctly
+ builds and installs and runs OK
+ scriptlets are sane
+ provides/requires/buildrequires look OK

Package is APPROVED.
Comment 4 Raphael Groner 2017-08-07 19:18:51 UTC 
https://pagure.io/dist-git-requests/issue/36
Comment 5 Ralph Bean 2017-08-10 15:30:52 UTC 
(fedrepo-req-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/cmatrix
Comment 6 Ralph Bean 2017-08-10 15:31:30 UTC 
(fedrepo-req-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/cmatrix
Comment 7 Raphael Groner 2017-08-13 00:12:29 UTC 
Imported into Rawhide. Will build for possible other branches when pagure allows so.
Note You need to log in before you can comment on or make changes to this bug.