Bug 2231393

Summary: Review Request: golang-modernc-libc - Runtime services for Go without cgo
Product: [Fedora] Fedora Reporter: W. Michael Petullo <mike>
Component: Package ReviewAssignee: Robert-André Mauchin 🐧 <zebob.m>
Status: ASSIGNED --- QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: package-review, zebob.m
Target Milestone: ---Flags: zebob.m: fedora-review?
mike: needinfo? (zebob.m)
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
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:
Bug Depends On: 2231392    
Bug Blocks: 2231928, 2231930, 2231933, 2231945    
Attachments:
Description Flags
The .spec file difference from Copr build 6266433 to 6299746 none

Description W. Michael Petullo 2023-08-11 13:25:11 UTC 
Spec URL: https://www.flyn.org/SRPMS/golang-modernc-libc.spec
SRPM URL: https://www.flyn.org/SRPMS/golang-modernc-libc-1.24.1-1.fc38.src.rpm
Description: Runtime services for Go without cgo
Fedora Account System Username: mikep

COPR repository:
[copr:copr.fedorainfracloud.org:mikep:golang]
name=Copr repo for golang owned by mikep
baseurl=https://download.copr.fedorainfracloud.org/results/mikep/golang/fedora-$releasever-$basearch/
type=rpm-md
skip_if_unavailable=True
gpgcheck=1
gpgkey=https://download.copr.fedorainfracloud.org/results/mikep/golang/pubkey.gpg
repo_gpgcheck=0
enabled=1
enabled_metadata=1
Comment 1 Robert-André Mauchin 🐧 2023-08-15 07:55:49 UTC 
Clean up this:

%global golicenses      LICENSE LICENSE-GO honnef.co/go/netdb/LICENSE\\\
                        musl/COPYRIGHT
%global godocs          AUTHORS CONTRIBUTORS README.md\\\
                        honnef.co/go/netdb/README.md musl/README

->

%global golicenses      LICENSE LICENSE-GO LICENSE-honnef\\\
                        COPYRIGHT-musl
%global godocs          AUTHORS CONTRIBUTORS README.md


and :


%license LICENSE LICENSE-GO honnef.co/go/netdb/LICENSE musl/COPYRIGHT
%doc AUTHORS CONTRIBUTORS README.md honnef.co/go/netdb/README.md musl/README

->

%license LICENSE LICENSE-GO LICENSE-honnef COPYRIGHT-musl
%doc AUTHORS CONTRIBUTORS README.md


with:

%prep
%goprep
%autopatch -p1
mv honnef.co/go/netdb/LICENSE LICENSE-honnef
mv musl/COPYRIGHT COPYRIGHT-musl




 - Add the license breakdown:

Name:           %{goname}
Release:        %autorelease
Summary:        C-runtime services for Go without cgo

# BSD-3-Clause: main program
# MIT: musl/ and honnef.co/
License:        BSD-3-Clause AND MIT
URL:            %{gourl}
Source:         %{gosource}


 - Do not glob the entire bindir/

%{_bindir}/libc


Besides, do you really need the binary for any project? Is it wise to include it?

Needinfo me when updated.
Comment 2 W. Michael Petullo 2023-08-15 17:19:01 UTC 
Spec URL: https://www.flyn.org/SRPMS/golang-modernc-libc.spec
SRPM URL: https://www.flyn.org/SRPMS/golang-modernc-libc-1.24.1-1.fc38.src.rpm

Followed suggestions above, and removed binary from package. How do I prevent the macros from building the binary package, which includes only documentation?
Comment 3 Fedora Review Service 2023-08-15 17:27:27 UTC 
Created attachment 1983424 [details]
The .spec file difference from Copr build 6266433 to 6299746