Spec URL: https://copr-be.cloud.fedoraproject.org/results/qulogic/git-lfs/fedora-rawhide-x86_64/00596933-git-lfs/git-lfs.spec SRPM URL: https://copr-be.cloud.fedoraproject.org/results/qulogic/git-lfs/fedora-rawhide-x86_64/00596933-git-lfs/git-lfs-2.2.1-2.fc28.src.rpm Description: Git Large File Storage (LFS) replaces large files such as audio samples, videos, datasets, and graphics with text pointers inside Git, while storing the file contents on a remote server. Fedora Account System Username: qulogic
*** Bug 1336168 has been marked as a duplicate of this bug. ***
Taking this review.
Initial pass through: > # Generate devel rpm > %global with_devel 0 Why is this disabled? Is it not possible to use Git LFS code as a module in other Go programs? > # Generate unit-test rpm > %global with_unit_test 0 Why is this disabled? > %post > /bin/git-lfs install --system Please use "%{_bindir}/%{name}" here, as that's how it is actually installed. > %preun > if [ $1 -gt 0 ]; then > /bin/git-lfs uninstall > fi > exit 0 Same here, and also properly indent the shell script here. > %files > %license LICENSE.md > %{_bindir}/%{name} > %doc %{_mandir}/man1/%{name}*.1* > %doc %{_mandir}/man5/%{name}*.5* It is redundant to declare man pages as documentation, as rpm auto-marks files installed into %{_mandir}/*/* as documentation files.
I can't say for certain whether you can or cannot use it as a module, but the draft Go spec doesn't even try to build devel/unit-test RPMs: https://fedoraproject.org/wiki/PackagingDrafts/Go#Packaging_a_binary Instead, I left those in there in case we need to enable them later.
According to godoc, there are no importers: https://godoc.org/github.com/git-lfs/git-lfs
Fixed the initial comments. Spec URL: https://copr-be.cloud.fedoraproject.org/results/qulogic/git-lfs/fedora-rawhide-x86_64/00597054-git-lfs/git-lfs.spec SRPM URL: https://copr-be.cloud.fedoraproject.org/results/qulogic/git-lfs/fedora-rawhide-x86_64/00597054-git-lfs/git-lfs-2.2.1-3.fc28.src.rpm Description: Git Large File Storage (LFS) replaces large files such as audio samples, videos, datasets, and graphics with text pointers inside Git, while storing the file contents on a remote server. Fedora Account System Username: qulogic
Package was generated through gofed, simplifying the review considerably. - Conforms to packaging guidelines (gofed generated spec) - license correct and valid - license file installed correctly - Binaries for applications installed - Scriptlets look sane PACKAGE APPROVED
to me scriptlets don't look sane... I would expect it to be executed while package is built, but this is one thing... Another thing could be done is to reference all files it creates via %ghost.
According to its man page, git only seems to support a single /etc/gitconfig file and not a directory like /etc/mercurial/hgrc.d. So I don't see how this could be done at build time. The file is not technically owned by git-lfs.
(fedrepo-req-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/git-lfs
(In reply to Igor Gnatenko from comment #8) > to me scriptlets don't look sane... I would expect it to be executed while > package is built, but this is one thing... > > Another thing could be done is to reference all files it creates via %ghost. Unfortunately, git is insane, so... :(
git-lfs-2.2.1-3.fc26 has been submitted as an update to Fedora 26. https://bodhi.fedoraproject.org/updates/FEDORA-2017-090bbadbe3
git-lfs-2.2.1-3.fc27 has been submitted as an update to Fedora 27. https://bodhi.fedoraproject.org/updates/FEDORA-2017-6ae6d44aaa
git-lfs-2.2.1-3.fc27 has been pushed to the Fedora 27 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2017-6ae6d44aaa
git-lfs-2.2.1-3.fc26 has been pushed to the Fedora 26 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2017-090bbadbe3
git-lfs-2.2.1-3.fc26 has been pushed to the Fedora 26 stable repository. If problems still persist, please make note of it in this bug report.
git-lfs-2.3.4-2.fc27 has been submitted as an update to Fedora 27. https://bodhi.fedoraproject.org/updates/FEDORA-2017-6ae6d44aaa
git-lfs-2.3.4-2.fc27 has been pushed to the Fedora 27 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2017-6ae6d44aaa
git-lfs-2.3.4-2.fc27 has been pushed to the Fedora 27 stable repository. If problems still persist, please make note of it in this bug report.
Only reopened because the freeze took so long.