[bug#36628] Minor fixes for pngcrush and python-aiorpcx

[bug#36628] Minor fixes for pngcrush and python-aiorpcx

[Arne Babenhauserheide]

Hi,

The following two patches are just fixing a typo and making guix lint happier.

Happy Hacking!
- Arne

[bug#36628] [PATCH 1/2] gnu: pngcrush: fix typo in description

[Arne Babenhauserheide]

* gnu/packages/image.scm (pngcrush): fix typo in description
---
 gnu/packages/image.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gnu/packages/image.scm b/gnu/packages/image.scm
index 511f5302b7..b465ff376e 100644
--- a/gnu/packages/image.scm
+++ b/gnu/packages/image.scm
@@ -250,7 +250,7 @@ APNG patch provides APNG support to libpng.")
       ("zlib" , zlib)))
    (home-page "https://pmt.sourceforge.io/pngcrush")
    (synopsis "Utility to compress PNG files")
-   (description "pngcrusqh is an optimizer for PNG (Portable Network Graphics)
+   (description "pngcrush is an optimizer for PNG (Portable Network Graphics)
 files.  It can compress them as much as 40% losslessly.")
    (license license:zlib)))

--
2.22.0

[bug#36628] [PATCH 1/2] gnu: pngcrush: fix typo in description

[Tobias Geerinckx-Rice]

[-- Attachment #1: Type: text/plain, Size: 1784 bytes --]

Arne,

Arne Babenhauserheide wrote:
> * gnu/packages/image.scm (pngcrush): fix typo in description
> ---
>  gnu/packages/image.scm | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/gnu/packages/image.scm b/gnu/packages/image.scm
> index 511f5302b7..b465ff376e 100644
> --- a/gnu/packages/image.scm
> +++ b/gnu/packages/image.scm
> @@ -250,7 +250,7 @@ APNG patch provides APNG support to 
> libpng.")
>        ("zlib" , zlib)))
>     (home-page "https://pmt.sourceforge.io/pngcrush")
>     (synopsis "Utility to compress PNG files")
> -   (description "pngcrusqh is an optimizer for PNG (Portable 
> Network Graphics)
> +   (description "pngcrush is an optimizer for PNG (Portable 
> Network Graphics)
>  files.  It can compress them as much as 40% losslessly.")
>     (license license:zlib)))

Thanks!  I was going to quickly apply these two before sleepytimes 
but noticed that your other patch changes the description (only) 
to begin with a capital letter, while this one remains lowercase. 
Is that deliberate?

I also couldn't help myself and kept editing...  What do you think 
of the following?  (It's still awfully short but that's another 
issue.)

--- a/gnu/packages/image.scm
+++ b/gnu/packages/image.scm
@@ -250,8 +250,8 @@ APNG patch provides APNG support to libpng.")
       ("zlib" , zlib)))
    (home-page "https://pmt.sourceforge.io/pngcrush")
    (synopsis "Utility to compress PNG files")
-   (description "pngcrusqh is an optimizer for PNG (Portable 
    Network Graphics)
-files.  It can compress them as much as 40% losslessly.")
+   (description "Pngcrush optimizes @acronym{PNG, Portable 
Network Graphics}
+images.  It can losslessly compress files by as much as an 
additional 40%.")
    (license license:zlib)))

Or something,

T G-R

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 227 bytes --]

[bug#36628] [PATCH 1/2] gnu: pngcrush: fix typo in description

[Arne Babenhauserheide]

[-- Attachment #1: Type: text/plain, Size: 2462 bytes --]


Tobias Geerinckx-Rice <me@tobias.gr> writes:

> Arne,
>
> Arne Babenhauserheide wrote:
>> * gnu/packages/image.scm (pngcrush): fix typo in description
>> ---
>>  gnu/packages/image.scm | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/gnu/packages/image.scm b/gnu/packages/image.scm
>> index 511f5302b7..b465ff376e 100644
>> --- a/gnu/packages/image.scm
>> +++ b/gnu/packages/image.scm
>> @@ -250,7 +250,7 @@ APNG patch provides APNG support to libpng.")
>>        ("zlib" , zlib)))
>>     (home-page "https://pmt.sourceforge.io/pngcrush")
>>     (synopsis "Utility to compress PNG files")
>> -   (description "pngcrusqh is an optimizer for PNG (Portable
>> Network Graphics)
>> +   (description "pngcrush is an optimizer for PNG (Portable Network
>> Graphics)
>>  files.  It can compress them as much as 40% losslessly.")
>>     (license license:zlib)))
>
> Thanks!  I was going to quickly apply these two before sleepytimes but
> noticed that your other patch changes the description (only) to begin
> with a capital letter, while this one remains lowercase. Is that
> deliberate?

For the other one guix lint complained, here it did not (maybe because
it started with exactly the package name?). That’s the only reason.

> I also couldn't help myself and kept editing...  What do you think of
> the following?  (It's still awfully short but that's another issue.)
>
> --- a/gnu/packages/image.scm
> +++ b/gnu/packages/image.scm
> @@ -250,8 +250,8 @@ APNG patch provides APNG support to libpng.")
>       ("zlib" , zlib)))
>    (home-page "https://pmt.sourceforge.io/pngcrush")
>    (synopsis "Utility to compress PNG files")
> -   (description "pngcrusqh is an optimizer for PNG (Portable  Network
> Graphics)
> -files.  It can compress them as much as 40% losslessly.")
> +   (description "Pngcrush optimizes @acronym{PNG, Portable Network
> Graphics}
> +images.  It can losslessly compress files by as much as an additional
> 40%.")
>    (license license:zlib)))

That sounds better. I think the correct English is "compress losslessly"
and not "losslessly compress", but aside from that it’s much more crisp.

I don’t think that being short is a problem, rather the opposite: if
it’s easy to read and provides the information needed, short is
often better.

So go for it! :)

Best wishes,
Arne
--
Unpolitisch sein
heißt politisch sein
ohne es zu merken

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 1076 bytes --]

[bug#36629] [PATCH 2/2] gnu: python-aiorpcx: capitalize description

[Arne Babenhauserheide]

* gnu/packages/python-web.scm (python-aiorpcx): capitalize description
---
 gnu/packages/python-web.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/python-web.scm b/gnu/packages/python-web.scm
index 35f471084f..e07951978e 100644
--- a/gnu/packages/python-web.scm
+++ b/gnu/packages/python-web.scm
@@ -168,8 +168,8 @@ using @url{https://github.com/saghul/pycares,pycares}.")
     (home-page "https://github.com/kyuupichan/aiorpcX")
     (synopsis "Generic asyncio RPC implementation")
     (description
-     "aiorpcX is a generic asyncio library implementation of RPC suitable for
-an application that is a client, server or both.
+     "The aiorpcX library is a generic asyncio implementation of RPC suitable
+for an application that is a client, server or both.

 The package includes a module with full coverage of JSON RPC versions 1.0 and
 2.0, JSON RPC protocol auto-detection, and arbitrary message framing.  It also
--
2.22.0

bug#36629: [PATCH 2/2] gnu: python-aiorpcx: capitalize description

[Ludovic Courtès]

Arne Babenhauserheide <arne_bab@web.de> skribis:

> * gnu/packages/python-web.scm (python-aiorpcx): capitalize description

Applied, thanks!