Bug 2430610

Summary: Review: Dog_Machine
Product: [Fedora] Fedora Reporter: carysayer
Component: Package ReviewAssignee: Nobody's working on this, feel free to take it <nobody>
Status: NEW --- QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 43CC: code, package-review
Target Milestone: ---   
Target Release: ---   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: ---
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Comment 1 Ben Beasley 2026-01-17 21:37:51 UTC
This looks pretty dubious. The upstream appears to be a one-time upload to the Internet Archive, https://archive.org/download/dog_machine /  https://archive.org/details/dog_machine. The upload to the IA claims it’s public-domain (“Public Domain Mark 1.0”), but the spec file lists the license as GPL-3.0-or-later.

I have not looked inside the source RPM at all, but I took a look at the spec file, and it is obviously just copying a pre-compiled executable in place. Everything in Fedora needs to be built from source[1]. There are other issues, like hard-coded /usr/lib instead of %{_libdir}[2], no effort to support primary architectures other than x86_64[3], odd installation of a C++ source file, missing debuginfo[4] (because the binary is not built from source), no source URL[5] without a good justification, and use of the Group tag[6].

It looks like this software was only published once, to the Internet Archive, and only with the express intent of including it in Fedora? Even if the intentions are good and genuine, that *looks* a little weird and even a bit suspicious, especially considering the submission contains a pre-compiled executable.

[1] https://docs.fedoraproject.org/en-US/packaging-guidelines/what-can-be-packaged/#prebuilt-binaries-or-libraries
[2] https://docs.fedoraproject.org/en-US/packaging-guidelines/#_macros
[3] https://docs.fedoraproject.org/en-US/packaging-guidelines/#_architecture_support
[4] https://docs.fedoraproject.org/en-US/packaging-guidelines/Debuginfo/#_missing_debuginfo_packages
[5] https://docs.fedoraproject.org/en-US/packaging-guidelines/SourceURL/
[6] https://docs.fedoraproject.org/en-US/packaging-guidelines/#_tags_and_sections

Comment 2 Fedora Review Service 2026-01-18 03:02:55 UTC
The ticket summary is not in the correct format.
Expected:

    Review Request: <main package name here> - <short summary here>

Found:

    Review: Dog_Machine

As a consequence, the package name cannot be parsed and submitted to
be automatically build. Please modify the ticket summary and trigger a
build by typing [fedora-review-service-build].


---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.