https://bugzilla.redhat.com/show_bug.cgi?id=1912706
Bug ID: 1912706 Summary: Review Request: ziglang - open-source programming language designed for robustness, optimality, and clarity Product: Fedora Version: rawhide Hardware: All OS: Linux Status: NEW Component: Package Review Severity: medium Priority: medium Assignee: nobody@fedoraproject.org Reporter: sguelton@redhat.com QA Contact: extras-qa@fedoraproject.org CC: package-review@lists.fedoraproject.org Target Milestone: --- Classification: Fedora
Spec URL: https://sergesanspaille.fedorapeople.org/ziglang.spec SRPM URL: https://sergesanspaille.fedorapeople.org/ziglang-0.7.1-1.src.rpm Description: Zig is an open-source programming language designed for robustness, optimality, and clarity Fedora Account System Username: sergesanspaille
https://bugzilla.redhat.com/show_bug.cgi?id=1912706
Stephen Kitt steve@sk2.org changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |steve@sk2.org Assignee|nobody@fedoraproject.org |steve@sk2.org Flags| |fedora-review?
https://bugzilla.redhat.com/show_bug.cgi?id=1912706
Stephen Kitt steve@sk2.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags| |needinfo?(sguelton@redhat.c | |om)
--- Comment #1 from Stephen Kitt steve@sk2.org --- A few initial comments:
* the description isn’t great; I think it should at least indicate that the package is a compiler, and describe the supported targets (and be line-wrapped) * the license isn’t complete, the various library headers have a lot of different licenses which affect the resulting binary package * the excluded arches need BZs, see https://docs.fedoraproject.org/en-US/packaging-guidelines/#_architecture_bui... * why isn’t the documentation generated?
https://bugzilla.redhat.com/show_bug.cgi?id=1912706
Petr Pisar ppisar@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED
https://bugzilla.redhat.com/show_bug.cgi?id=1912706
serge_sans_paille sguelton@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags|needinfo?(sguelton@redhat.c | |om) |
--- Comment #2 from serge_sans_paille sguelton@redhat.com --- Thanks for the review and sorry for the late answer. I've uploaded the updated .spec and .rpms * description: done * I've updated the license field, including the comments needed for each bundled lib * concerning the excluded archs, I should open the BZ only once the package is accepted * there's... no documentation to generate, unless I missed something in their code source ;-)
https://bugzilla.redhat.com/show_bug.cgi?id=1912706
--- Comment #3 from Stephen Kitt steve@sk2.org --- Regarding the docs, I was thinking of the generator in the doc/ directory. It would be worth at least filing a bug upstream to ask for it to be integrated into the build ;-).
https://bugzilla.redhat.com/show_bug.cgi?id=1912706
--- Comment #4 from serge_sans_paille sguelton@redhat.com --- Issue opened as https://github.com/ziglang/zig/issues/7897
https://bugzilla.redhat.com/show_bug.cgi?id=1912706
--- Comment #5 from serge_sans_paille sguelton@redhat.com --- Spec and rpms updated with documentation generated. I've also submitted a patch upstream to automate documentation build (see https://github.com/ziglang/zig/pull/7911)
https://bugzilla.redhat.com/show_bug.cgi?id=1912706
Jan Drögehoff sentrycraft123@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |sentrycraft123@gmail.com
--- Comment #6 from Jan Drögehoff sentrycraft123@gmail.com --- you can simply build documentation with 'zig build docs' using the freshly compiled zig binary
am not a fan of bundling the zig stdlib with the actual binary since the stdlib is distributed in source form making it arch independent this would also allow the main zig package to be licensed under just MIT
The generated langref.html documentation is also a handfull so I suggest making a -doc package for it
since zig has no official manpage yet (https://github.com/ziglang/zig/issues/715) I suggest using help2man
here is the spec I have been maintaining for a copr repo for reference to my suggestions https://pagure.io/zig-spec/blob/fedora/f/zig.spec
https://bugzilla.redhat.com/show_bug.cgi?id=1912706
--- Comment #7 from serge_sans_paille sguelton@redhat.com --- Thanks for all the comments. Spec file updated here
Spec URL: https://sergesanspaille.fedorapeople.org/zig.spec SRPM URL: https://sergesanspaille.fedorapeople.org/zig-0.7.1-1.src.rpm
scratch-build: https://koji.fedoraproject.org/koji/taskinfo?taskID=61017957
https://bugzilla.redhat.com/show_bug.cgi?id=1912706
Stephen Kitt steve@sk2.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags| |needinfo?(sguelton@redhat.c | |om)
--- Comment #8 from Stephen Kitt steve@sk2.org --- We’re getting there, thanks; the remaining issues are:
* zig-doc ships langref.html in /usr/share/doc/zig instead of /usr/share/doc/zig-doc * the zig-doc description results in a line that’s too long; it should be wrapped after “%{name}.” * the zig-libs summary isn’t capitalised (“zig standard library”) * /lib/ is hard-coded (“%{_prefix}/lib/%{name}”) * “optimality” isn’t recognised as a word (but it is in at least some dictionaries)
Also, Zig 0.7.1 requires clang 11 but Rawhide now has 12.
https://bugzilla.redhat.com/show_bug.cgi?id=1912706
--- Comment #9 from Stephen Kitt steve@sk2.org --- To build with clang 11, change the cmake invocation to
%cmake \ -DLLVM_CONFIG_EXE=%{_bindir}/llvm-config-11-%{__isa_bits} \ -DLLVM_INCLUDE_DIRS=%{_libdir}/llvm11/include \ -DLLVM_LIBRARIES=%{_libdir}/llvm11/lib \ -DCLANG_INCLUDE_DIRS=%{_libdir}/include/clang \ -DCLANG_LIBRARIES=%{_libdir}/llvm11/lib \ -DCMAKE_BUILD_TYPE=RelWithDebInfo \ -DZIG_PREFER_CLANG_CPP_DYLIB=ON \ -DZIG_VERSION:STRING="%{version}"
and the build requires to
BuildRequires: llvm11-devel BuildRequires: clang11-devel
The overall build fails because the lld11 libraries aren’t available.
https://bugzilla.redhat.com/show_bug.cgi?id=1912706
--- Comment #10 from Jan Drögehoff sentrycraft123@gmail.com --- (In reply to Stephen Kitt from comment #8)
- /lib/ is hard-coded (“%{_prefix}/lib/%{name}”)
Last I checked zig does not accept lib64 or the like and considering the entire standard library is distributed in plain text making an arch independent package that puts it into /lib/ seems fine to me
https://bugzilla.redhat.com/show_bug.cgi?id=1912706
--- Comment #11 from Stephen Kitt steve@sk2.org --- (In reply to Jan Drögehoff from comment #10)
(In reply to Stephen Kitt from comment #8)
- /lib/ is hard-coded (“%{_prefix}/lib/%{name}”)
Last I checked zig does not accept lib64 or the like and considering the entire standard library is distributed in plain text making an arch independent package that puts it into /lib/ seems fine to me
Thanks for the explanation, I agree that seems fine.
package-review@lists.fedoraproject.org