all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de>
To: "Ludovic Courtès" <ludo@gnu.org>
Cc: guix-devel <guix-devel@gnu.org>
Subject: Re: [PATCH] Add Pandoc (and whatever it needs)
Date: Thu, 10 Dec 2015 14:43:40 +0100	[thread overview]
Message-ID: <idjmvti8lmb.fsf@bimsb-sys02.mdc-berlin.net> (raw)
In-Reply-To: <874mfrl2xl.fsf@gnu.org>


Ludovic Courtès <ludo@gnu.org> writes:

>> +    (home-page "https://github.com/haskell-crypto/cryptonite")
>> +    (synopsis "Cryptography primitives")
>> +    (description
>> +     "This package is a repository of cryptographic primitives for Haskell.
>> +It strives to be a cryptographic kitchen sink that provides cryptography for
>> +everyone.
>> +
>> +Supported symmetric ciphers: AES, DES, 3DES, Blowfish, Camellia, RC4, Salsa,
>> +ChaCha; supported hash functions: SHA1, SHA2, SHA3, MD2, MD4, MD5, Keccak,
>> +Skein, Ripemd, Tiger, Whirlpool, Blake2; MAC: HMAC, Poly1305; assymmetric
>> +crypto: DSA, RSA, DH, ECDH, ECDSA, ECC, Curve25519, Ed25519; key derivation
>> +functions: PBKDF2, Scrypt; cryptographic random number generation: system
>> +entropy, deterministic random generator; data-related features:
>> +@dfn{anti-forensic information splitter} (AFIS).")
>
> What about something like “It supports a wide range of symmetric
> ciphers, cryptographic hash functions, public key algorithms, key
> derivation numbers, cryptographic random number generators, and more.”?

I jumped over this comment as I applied the suggestions, so I updated
the description in a follow-up commit.  My apologies for the noise!

>> +  (home-page "http://github.com/vincenthz/hs-tls")
>> +  (synopsis
>> +    "TLS/SSL protocol native implementation (Server and Client)")
>> +  (description
>> +    "Native Haskell TLS and SSL protocol implementation for server and client. . This provides a high-level implementation of a sensitive security protocol, eliminating a common set of security issues through the use of the advanced type system, high level constructions and common Haskell features. . Currently implement the SSL3.0, TLS1.0, TLS1.1 and TLS1.2 protocol, and support RSA and Ephemeral (Elliptic curve and regular) Diffie Hellman key exchanges, and many extensions. . Some debug tools linked with tls, are available through the <http://hackage.haskell.org/package/tls-debug/>.")
>
> Could you wrap it, remove extra periods, and use @url?

Ouch!  This was completely untouched after importing from Hackage.  I
cleaned up the formatting of the package expression and also fixed the
description.

>> +    (home-page "http://pandoc.org")
>> +    (synopsis "Conversion between markup formats")
>> +    (description
>> +     "Pandoc is a Haskell library for converting from one markup format to
>> +another, and a command-line tool that uses this library.  It can read markdown
>> +and (subsets of) HTML, reStructuredText, LaTeX, DocBook, MediaWiki markup,
>> +TWiki markup, Haddock markup, OPML, Emacs Org-Mode, txt2tags, Word Docx, ODT,
>> +and Textile, and it can write Markdown, reStructuredText, XHTML, HTML 5,
>> +LaTeX, ConTeXt, DocBook, OPML, OpenDocument, ODT, Word docx, RTF, MediaWiki,
>> +DokuWiki, Textile, groff man pages, plain text, Emacs Org-Mode, AsciiDoc,
>> +Haddock markup, EPUB (v2 and v3), FictionBook2, InDesign ICML, and several
>> +kinds of HTML/javascript slide shows (S5, Slidy, Slideous, DZSlides,
>> +reveal.js).
>> +
>> +Pandoc extends standard markdown syntax with footnotes, embedded LaTeX,
>> +definition lists, tables, and other features.  A compatibility mode is
>> +provided for those who need a drop-in replacement for Markdown.pl.  In
>> +contrast to existing tools for converting markdown to HTML, which use regex
>> +substitutions, pandoc has a modular design: it consists of a set of readers,
>> +which parse text in a given format and produce a native representation of the
>> +document, and a set of writers, which convert this native representation into
>> +a target format.  Thus, adding an input or output format requires only adding
>> +a reader or writer.")
>
> Would be nice to keep just one third of it, notably by omitting the list
> of supported formats.  :-)

I shortened it substantially and pushed the modified commits.

Thank you for what must have been a very tedious review!

~~ Ricardo

      reply	other threads:[~2015-12-10 13:44 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-11-27 12:50 [PATCH] Add Pandoc (and whatever it needs) Ricardo Wurmus
2015-11-28 20:27 ` Leo Famulari
2015-12-09 21:36 ` Ludovic Courtès
2015-12-10 13:43   ` Ricardo Wurmus [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=idjmvti8lmb.fsf@bimsb-sys02.mdc-berlin.net \
    --to=ricardo.wurmus@mdc-berlin.de \
    --cc=guix-devel@gnu.org \
    --cc=ludo@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.