[bug#31949] [PATCH] gnu: Add docx2txt.

[ludo@gnu.org (Ludovic Courtès)]

Hi,

Pierre Neidhardt <ambrevar@gmail.com> skribis:

> * gnu/packages/textutils.scm (docx2txt): New variable.

[...]

> +    (source (origin
> +              (method url-fetch)
> +              (uri (string-append
> +                    "http://downloads.sourceforge.net/docx2txt/docx2txt-"
> +                    version ".tgz"))

Could you use mirror://sourceforge?

> +       #:make-flags (list (string-append "BINDIR=" (assoc-ref %outputs "out") "/bin")
> +                          (string-append "CONFIGDIR=" (assoc-ref %outputs "out") "/etc")

Lines are a bit long.  :-)

> +    (synopsis "Recover text from .docx files, with good formatting")

@file{.docx} please.

> +    (description
> +     "@command{docx2txt} is a perl based command line utility to convert

s/perl/Perl/

> +Microsoft Office™ .docx documents to equivalent text documents. Latest version

No need for the trademark sign; two spaces after period.

> +@itemize
> +@item Character conversions (\" ' < & > -, fractions and some mathematical
> +symbols, etc.); currency characters are converted to respective names like
> +Euro.

Maybe you remove what’s in parentheses?  Or use @code.

Could you send an updated patch?  Make sure ‘guix lint’ is happy.  :-)

Thanks,
Ludo’.