Bug 1460246

>Patch1:         lua_package.patch
While the patch itself is rather straight-forward, it'd still be nice to have a short comment in the spec file explaining what it does.

>Summary:        (...) done.
>Group:          Applications/Productivity
The Summary should not end with a dot and the Group tag should not be used at all.
https://fedoraproject.org/wiki/Packaging:Guidelines#Tags_and_Sections

>make -j
I believe you should use the %{?_smp_mflags} macro here (it expands to the -j option).

>make install PREFIX=$RPM_BUILD_ROOT/usr
Use the %{_prefix} macro instead of "/usr".

>%_mandir/man1/wordgrinder.1.gz
You should not assume that gzip will be used for compressing manpages - instead, use a pattern that can match both compressed and non-compressed files.
https://fedoraproject.org/wiki/Packaging:Guidelines#Manpages

>* Wed Jun 07 2017 Ben Cotton <bcotton> - 0.6-1
Probably a typo in the domain name ("porject" instead of "project").

Also, there are some rpmlint errors:
wordgrinder.x86_64: E: explicit-lib-dependency libXft
wordgrinder.x86_64: E: explicit-lib-dependency zlib
wordgrinder.x86_64: E: summary-too-long

Thanks for the quick review. I believe I've addressed all of your concerns. Spec and SRPM updated:

Spec URL: https://bcotton.fedorapeople.org/wordgrinder/wordgrinder.spec
SRPM URL: https://bcotton.fedorapeople.org/wordgrinder/wordgrinder-0.6-1.fc25.src.rpm

Updated again to address upstream's concerns with the patch:

Spec: https://bcotton.fedorapeople.org/wordgrinder/wordgrinder.spec
SRPM: https://bcotton.fedorapeople.org/wordgrinder/wordgrinder-0.6-2.fc25.src.rpm

Created attachment 1294931 [details]
Build log

Fails to build on f26.

Ah! Looks like luajit changed versions. I've reworked it a little bit and it now builds in f25 and f26 mocks:

Spec: https://bcotton.fedorapeople.org/wordgrinder/wordgrinder.spec
SRPM: https://bcotton.fedorapeople.org/wordgrinder/wordgrinder-0.6-3.fc25.src.rpm

Much better.

- rpmlint checks return:

wordgrinder-debuginfo.x86_64: E: debuginfo-without-sources
This debuginfo package appears to contain debug symbols but no source files.
This is often a sign of binaries being unexpectedly stripped too early during
the build, or being compiled without compiler debug flags (which again often
is a sign of distro's default compiler flags ignored which might have security
consequences), or other compiler flags which result in rpmbuild's debuginfo
extraction not working as expected.  Verify that the binaries are not
unexpectedly stripped and that the intended compiler flags are used.


- package meets naming guidelines
- package meets packaging guidelines
- license ( MIT ) OK, text in %doc, matches source
- spec file legible, in am. english
- source matches upstream
- package compiles on devel (x86)
- no missing BR
- no unnecessary BR
- no locales
- not relocatable
- owns all directories that it creates
- no duplicate files
- permissions ok
- %clean ok
- macro use consistent
- code, not content
- no need for -docs
- nothing in %doc affects runtime
- no need for .desktop file

It would be good to break up the BuildRequires into multiple lines, for readability.

So at the moment the debuginfo is the only fix needed.

I broke up the BuildRequires as suggested and modified my patch to correct the debuginfo issues.

rpmlint now returns:
wordgrinder-debuginfo.x86_64: W: hidden-file-or-dir /usr/src/debug/wordgrinder-wordgrinder-0.6/.obj
wordgrinder-debuginfo.x86_64: W: hidden-file-or-dir /usr/src/debug/wordgrinder-wordgrinder-0.6/.obj
1 packages and 0 specfiles checked; 0 errors, 2 warnings.


The hidden file is expected. Updated SRPM/spec:

Spec: https://bcotton.fedorapeople.org/wordgrinder/wordgrinder.spec
SRPM: https://bcotton.fedorapeople.org/wordgrinder/wordgrinder-0.6-4.fc25.src.rpm

Excellent.  There's also:

wordgrinder.src:15: W: mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 15)

but you can finish that at import.

APPROVED.

Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/rpms/wordgrinder