Bug 1810897 - Review Request: hello - Prints a familiar, friendly greeting
Summary: Review Request: hello - Prints a familiar, friendly greeting
Keywords:
Status: CLOSED ERRATA
Alias: None
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:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2020-03-06 07:01 UTC by Jens Petersen
Modified: 2020-04-25 02:19 UTC (History)
3 users (show)

Fixed In Version: hello-2.10-3.fc33
Clone Of:
Environment:
Last Closed: 2020-04-16 22:40:47 UTC
Type: ---
Embargoed:
zbyszek: fedora-review+


Attachments (Terms of Use)

Description Jens Petersen 2020-03-06 07:01:03 UTC
Spec URL: https://petersen.fedorapeople.org/reviews/hello/hello.spec
SRPM URL: https://petersen.fedorapeople.org/reviews/hello/hello-2.10-1.fc32.src.rpm

Description:
The GNU Hello program produces a familiar, friendly greeting.
Yes, this is another implementation of the classic program that
prints âHello, world!â when you run it.

However, unlike the minimal version often seen, GNU Hello processes
its argument list to modify its behavior, supports greetings in many
languages, and so on. The primary purpose of GNU Hello is to
demonstrate how to write other programs that do these things; it
serves as a model for GNU coding standards and GNU maintainer
practices.


Koji scratch build: https://koji.fedoraproject.org/koji/taskinfo?taskID=42240531

Comment 2 Jens Petersen 2020-03-06 07:44:02 UTC
This package was retired in 2012 (see bug 821762).
The original review was bug 476936 (2009).

A fork is here: https://src.fedoraproject.org/fork/petersen/rpms/hello

https://koji.fedoraproject.org/koji/taskinfo?taskID=42240809

Comment 3 Zbigniew Jędrzejewski-Szmek 2020-03-06 14:28:29 UTC
> URL:            http://www.gnu.org/software/hello/

https please. Even GNU supports that nowadays.

make %{?_smp_mflags} → %make_build

make install DESTDIR=$RPM_BUILD_ROOT → %make_install

Comment 4 Zbigniew Jędrzejewski-Szmek 2020-03-06 14:31:21 UTC
$RPM_BUILD_ROOT → %buildroot to make things more modern.

Comment 5 Zbigniew Jędrzejewski-Szmek 2020-03-06 14:36:36 UTC
- If (and only if) 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 %license.
  Note: License file COPYING is not marked as %license
  See: https://docs.fedoraproject.org/en-US/packaging-
  guidelines/LicensingGuidelines/#_license_text

Also: Requires: info seems suspicious.

Comment 6 Jens Petersen 2020-04-01 14:18:23 UTC
Thank you, Zbigniew, sorry for the delay.
I think I have fixed all the issues you pointed out:



(In reply to Zbigniew Jędrzejewski-Szmek from comment #3)
> > URL:            http://www.gnu.org/software/hello/
> 
> https please. Even GNU supports that nowadays.

:-)

> make %{?_smp_mflags} → %make_build
> make install DESTDIR=$RPM_BUILD_ROOT → %make_install
(In reply to Zbigniew Jędrzejewski-Szmek from comment #4)
> $RPM_BUILD_ROOT → %buildroot to make things more modern.

Done

(In reply to Zbigniew Jędrzejewski-Szmek from comment #5)
>   Note: License file COPYING is not marked as %license

Oops, good catch

> Also: Requires: info seems suspicious.

How about Recommends?


https://src.fedoraproject.org/fork/petersen/rpms/hello/c/1317b3590753f0eea349c0a266fb51fa4a5e0d70

Comment 7 Jens Petersen 2020-04-01 14:21:37 UTC
(In reply to Jens Petersen from comment #6)
> https://src.fedoraproject.org/fork/petersen/rpms/hello/c/
> 1317b3590753f0eea349c0a266fb51fa4a5e0d70

Okay, now with a release bump:
https://src.fedoraproject.org/fork/petersen/rpms/hello/c/4ff214be2e5e0b252e2d42e3ea81baf04183e5cd

Comment 8 Zbigniew Jędrzejewski-Szmek 2020-04-01 14:25:55 UTC
Looks all good now. Package is RE-APPROVED.

Comment 10 Jens Petersen 2020-04-02 06:40:31 UTC
I requested unretirement to releng in https://pagure.io/releng/issue/9377

Comment 11 Fedora Update System 2020-04-08 06:14:54 UTC
FEDORA-2020-f0bfd67c52 has been submitted as an update to Fedora 32. https://bodhi.fedoraproject.org/updates/FEDORA-2020-f0bfd67c52

Comment 12 Fedora Update System 2020-04-08 10:04:13 UTC
FEDORA-2020-48b58104ed has been submitted as an update to Fedora 31. https://bodhi.fedoraproject.org/updates/FEDORA-2020-48b58104ed

Comment 13 Fedora Update System 2020-04-08 11:51:22 UTC
FEDORA-2020-0348f28a44 has been submitted as an update to Fedora 30. https://bodhi.fedoraproject.org/updates/FEDORA-2020-0348f28a44

Comment 14 Fedora Update System 2020-04-09 18:37:50 UTC
FEDORA-2020-f0bfd67c52 has been pushed to the Fedora 32 testing repository.
In short time you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing --advisory=FEDORA-2020-f0bfd67c52 \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2020-f0bfd67c52

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 15 Fedora Update System 2020-04-09 19:13:41 UTC
FEDORA-2020-48b58104ed has been pushed to the Fedora 31 testing repository.
In short time you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing --advisory=FEDORA-2020-48b58104ed \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2020-48b58104ed

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 16 Fedora Update System 2020-04-09 20:14:17 UTC
FEDORA-2020-0348f28a44 has been pushed to the Fedora 30 testing repository.
In short time you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing --advisory=FEDORA-2020-0348f28a44 \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2020-0348f28a44

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 17 Fedora Update System 2020-04-16 22:40:47 UTC
FEDORA-2020-48b58104ed has been pushed to the Fedora 31 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 18 Fedora Update System 2020-04-18 02:31:33 UTC
FEDORA-2020-0348f28a44 has been pushed to the Fedora 30 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 19 Fedora Update System 2020-04-25 02:19:05 UTC
FEDORA-2020-f0bfd67c52 has been pushed to the Fedora 32 stable repository.
If problem still persists, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.