[Bug 1987564] Review Request: <mongoInc-backup-0.1-1.fc34.x86_64.rpm> - <mongoInc-backup>

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1987564

Artur Frenszek-Iwicki <fedora@xxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |fedora@xxxxxxxxxx



--- Comment #3 from Artur Frenszek-Iwicki <fedora@xxxxxxxxxx> ---
> Spec URL: https://github.com/OEntegrasyon/MongoDB_incremental_backup/releases/tag/mongoInc-backup
> SRPM URL: https://github.com/OEntegrasyon/MongoDB_incremental_backup/releases/tag/mongoInc-backup
This is a link to a GitHub downloads page. Please use direct links to the "raw"
.spec and .src.rpm files.

> License:      GNU
GNU is a free software project, not the name of a license. Looking at the
sources, you meant the GNU General Public License, version 3. Use "GPLv3" here.
https://fedoraproject.org/wiki/Licensing:Main?rd=Licensing#SoftwareLicenses

> Group:        System
> Packager:   Ergün Elvan Bilsel
Not used in Fedora.
https://docs.fedoraproject.org/en-US/packaging-guidelines/#_tags_and_sections

> mkdir -p $RPM_BUILD_ROOT/etc/mongobackup/
> mkdir -p $RPM_BUILD_ROOT/usr/local/bin/
> mkdir -p $RPM_BUILD_ROOT/usr/share/licenses/mongoInc-backup/
> mkdir -p $RPM_BUILD_ROOT/usr/share/doc/mongoInc-backup/
Do not use hard-coded paths. You should use the appropriate RPM macros instead,
e.g. '/usr/share' -> '%{_datadir}'.
https://docs.fedoraproject.org/en-US/packaging-guidelines/RPMMacros/#_macros_for_paths_set_and_used_by_build_systems

> cp /home/ergunbilsel/PycharmProjects/mongoinc/mongoInc-backup-0.1/mongodb-backup.config $RPM_BUILD_ROOT/etc/mongobackup/
> ...
> cp /home/ergunbilsel/PycharmProjects/mongoinc/mongoInc-backup-0.1/THANKS  $RPM_BUILD_ROOT/usr/share/doc/mongoInc-backup/
1. Uh... this will fail when run anywhere else apart from your machine. 
2. This is done in %prep, whereas it should be done in %install.


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux