[Bug 2218338] Review Request: skribilo - A free document production tool

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

 



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

Andrew Bauer <zonexpertconsulting@xxxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
              Flags|                            |fedora-review?
                 CC|                            |zonexpertconsulting@outlook
                   |                            |.com



--- Comment #6 from Andrew Bauer <zonexpertconsulting@xxxxxxxxxxx> ---
Wow, you've been waiting awhile for this one. There are elements of this
specfile I am not familiar with (i.e. lang files), but I'll get up to speed as
we progress.

I haven't run fedora-review yet. For starters let's discuss rpmlint, since it
was rather noisy:


$ rpmlint *.rpm
============================ rpmlint session starts
============================
rpmlint: 2.4.0
configuration:
    /usr/lib/python3.11/site-packages/rpmlint/configdefaults.toml
    /etc/xdg/rpmlint/fedora-legacy-licenses.toml
    /etc/xdg/rpmlint/fedora-spdx-licenses.toml
    /etc/xdg/rpmlint/fedora.toml
    /etc/xdg/rpmlint/scoring.toml
    /etc/xdg/rpmlint/users-groups.toml
    /etc/xdg/rpmlint/warn-on-functions.toml
checks: 31, packages: 2

skribilo.noarch: E: zero-length /usr/share/doc/skribilo/ChangeLog
skribilo.noarch: W: only-non-binary-in-usr-lib
skribilo.noarch: E: noarch-with-lib64
skribilo.noarch: W: no-manual-page-for-binary skribilo-config
skribilo.spec:9: W: mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 9)
skribilo.spec:147: W: libdir-macro-in-noarch-package (main package)
%{_libdir}/guile/%{mver}/site-ccache/diff.go
skribilo.spec:148: W: libdir-macro-in-noarch-package (main package)
%{_libdir}/guile/%{mver}/site-ccache/%{name}.go
skribilo.spec:149: W: libdir-macro-in-noarch-package (main package) %dir
%{_libdir}/guile/%{mver}/site-ccache/%{name}
skribilo.spec:150: W: libdir-macro-in-noarch-package (main package)
%{_libdir}/guile/%{mver}/site-ccache/%{name}/*.go
skribilo.spec:151: W: libdir-macro-in-noarch-package (main package) %dir
%{_libdir}/guile/%{mver}/site-ccache/%{name}/biblio
skribilo.spec:152: W: libdir-macro-in-noarch-package (main package)
%{_libdir}/guile/%{mver}/site-ccache/%{name}/biblio/*.go
skribilo.spec:153: W: libdir-macro-in-noarch-package (main package) %dir
%{_libdir}/guile/%{mver}/site-ccache/%{name}/documentation
skribilo.spec:154: W: libdir-macro-in-noarch-package (main package)
%{_libdir}/guile/%{mver}/site-ccache/%{name}/documentation/*.go
skribilo.spec:155: W: libdir-macro-in-noarch-package (main package) %dir
%{_libdir}/guile/%{mver}/site-ccache/%{name}/engine
skribilo.spec:156: W: libdir-macro-in-noarch-package (main package)
%{_libdir}/guile/%{mver}/site-ccache/%{name}/engine/*.go
skribilo.spec:157: W: libdir-macro-in-noarch-package (main package) %dir
%{_libdir}/guile/%{mver}/site-ccache/%{name}/package
skribilo.spec:158: W: libdir-macro-in-noarch-package (main package)
%{_libdir}/guile/%{mver}/site-ccache/%{name}/package/*.go
skribilo.spec:159: W: libdir-macro-in-noarch-package (main package) %dir
%{_libdir}/guile/%{mver}/site-ccache/%{name}/package/eq
skribilo.spec:160: W: libdir-macro-in-noarch-package (main package)
%{_libdir}/guile/%{mver}/site-ccache/%{name}/package/eq/*.go
skribilo.spec:161: W: libdir-macro-in-noarch-package (main package) %dir
%{_libdir}/guile/%{mver}/site-ccache/%{name}/package/pie
skribilo.spec:162: W: libdir-macro-in-noarch-package (main package)
%{_libdir}/guile/%{mver}/site-ccache/%{name}/package/pie/*.go
skribilo.spec:163: W: libdir-macro-in-noarch-package (main package) %dir
%{_libdir}/guile/%{mver}/site-ccache/%{name}/package/slide
skribilo.spec:164: W: libdir-macro-in-noarch-package (main package)
%{_libdir}/guile/%{mver}/site-ccache/%{name}/package/slide/*.go
skribilo.spec:165: W: libdir-macro-in-noarch-package (main package) %dir
%{_libdir}/guile/%{mver}/site-ccache/%{name}/reader
skribilo.spec:166: W: libdir-macro-in-noarch-package (main package)
%{_libdir}/guile/%{mver}/site-ccache/%{name}/reader/*.go
skribilo.spec:167: W: libdir-macro-in-noarch-package (main package) %dir
%{_libdir}/guile/%{mver}/site-ccache/%{name}/source
skribilo.spec:168: W: libdir-macro-in-noarch-package (main package)
%{_libdir}/guile/%{mver}/site-ccache/%{name}/source/*.go
skribilo.spec:169: W: libdir-macro-in-noarch-package (main package) %dir
%{_libdir}/guile/%{mver}/site-ccache/%{name}/utils
skribilo.spec:170: W: libdir-macro-in-noarch-package (main package)
%{_libdir}/guile/%{mver}/site-ccache/%{name}/utils/*.go
skribilo.noarch: W: file-not-utf8 /usr/share/doc/skribilo/ChangeLog.Skribe
skribilo.noarch: W: file-not-utf8 /usr/share/man/man1/skribilo.1.gz
skribilo.noarch: W: file-not-in-%lang
/usr/share/locale/da/LC_MESSAGES/skribilo.mo
skribilo.noarch: W: file-not-in-%lang
/usr/share/locale/es/LC_MESSAGES/skribilo.mo
skribilo.noarch: W: file-not-in-%lang
/usr/share/locale/fr/LC_MESSAGES/skribilo.mo
skribilo.noarch: W: file-not-in-%lang
/usr/share/locale/fur/LC_MESSAGES/skribilo.mo
skribilo.noarch: W: file-not-in-%lang
/usr/share/locale/hu/LC_MESSAGES/skribilo.mo
skribilo.noarch: W: file-not-in-%lang
/usr/share/locale/pt_BR/LC_MESSAGES/skribilo.mo
skribilo.noarch: W: file-not-in-%lang
/usr/share/locale/sr/LC_MESSAGES/skribilo.mo
skribilo.noarch: W: file-not-in-%lang
/usr/share/locale/sv/LC_MESSAGES/skribilo.mo
skribilo.noarch: W: file-not-in-%lang
/usr/share/locale/uk/LC_MESSAGES/skribilo.mo
skribilo.noarch: W: devel-file-in-non-devel-package /usr/bin/skribilo-config
skribilo.noarch: E: description-line-too-long Last but not least, Skribilo can
be thought of as a complete document programming
skribilo.noarch: E: description-line-too-long framework for the Scheme
programming language that may be used to automate a variety
skribilo.noarch: E: description-line-too-long of document generation tasks.
Technically, the Skribilo language/API is an embedded
skribilo.noarch: E: description-line-too-long domain-specific language (EDSL),
implemented via so-called “deep embedding”. Skribilo
skribilo.src: E: description-line-too-long Last but not least, Skribilo can be
thought of as a complete document programming
skribilo.src: E: description-line-too-long framework for the Scheme programming
language that may be used to automate a variety
skribilo.src: E: description-line-too-long of document generation tasks.
Technically, the Skribilo language/API is an embedded
skribilo.src: E: description-line-too-long domain-specific language (EDSL),
implemented via so-called “deep embedding”. Skribilo
skribilo.noarch: W: dangling-relative-symlink /usr/share/info/bsd.png.gz
../doc/skribilo/bsd.png.gz
skribilo.noarch: W: dangling-relative-symlink
/usr/share/info/document-toolchain.png.gz
../doc/skribilo/document-toolchain.png.gz
skribilo.noarch: W: dangling-relative-symlink
/usr/share/info/eq-disponibilite.png.gz ../doc/skribilo/eq-disponibilite.png.gz
skribilo.noarch: W: dangling-relative-symlink
/usr/share/info/eq-limit-b-over-l-simplified.png.gz
../doc/skribilo/eq-limit-b-over-l-simplified.png.gz
skribilo.noarch: W: dangling-relative-symlink
/usr/share/info/eq-limit-b-over-l.png.gz
../doc/skribilo/eq-limit-b-over-l.png.gz
skribilo.noarch: W: dangling-relative-symlink /usr/share/info/linux.png.gz
../doc/skribilo/linux.png.gz
 2 packages and 0 specfiles checked; 10 errors, 45 warnings, 10 badness; has
taken 0.4 s 

============================ rpmlint session ends ============================

TO-DO:
- please remove the zerolength ChangeLog or consider symlinking it with
ChangeLog.Skribe
- I know rpmlint is nitpicking, but please use either spaces or tabs in the
specfile, not both
- please shorten the length of each line in the description such that each is
less than 80 characters

- While skribilo itself is just a shell script and hence, not arch dependent,
the .go files generated by guile are architecture dependent. We need to do
something about this. Perhaps the simplest thing to do would be to remove the
noarch flag in the specfile.

- Rpmlint is complaining about the lang files placed under /usr/share/locale.
This implies we should be doing something more than just placing files there.
I'm not sure yet what the answer is to this, but I did find mention of this
%find_lang macro:
https://docs.fedoraproject.org/en-US/packaging-guidelines/#_why_do_we_need_to_use_find_lang

Comments:
I haven't yet verified the relative symlinks rpmlint is complaining about, but
I suspect they are correct.
The rest of the warnings can be ignored.


-- 
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
https://bugzilla.redhat.com/show_bug.cgi?id=2218338

Report this comment as SPAM: https://bugzilla.redhat.com/enter_bug.cgi?product=Bugzilla&format=report-spam&short_desc=Report%20of%20Bug%202218338%23c6
_______________________________________________
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, report it: https://pagure.io/fedora-infrastructure/new_issue




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

  Powered by Linux