Re: [PATCH 1/3] gnu: Add utfcpp.

2016-05-12 Thread Alex Kost
Alex Griffin (2016-05-11 02:31 +0300) wrote: > Here's an updated patch with utfcpp moved to textutils. > > From 0fada0c526a9d3aa1573d96ec488da1c20b43a0d Mon Sep 17 00:00:00 2001 > From: Alex Griffin > Date: Sat, 7 May 2016 12:16:39 -0500 > Subject: [PATCH 1/2] gnu: Add utfcpp. > > * gnu/packages/

Re: [PATCH 1/3] gnu: Add utfcpp.

2016-05-10 Thread Alex Griffin
Here's an updated patch with utfcpp moved to textutils. -- Alex Griffin From 0fada0c526a9d3aa1573d96ec488da1c20b43a0d Mon Sep 17 00:00:00 2001 From: Alex Griffin Date: Sat, 7 May 2016 12:16:39 -0500 Subject: [PATCH 1/2] gnu: Add utfcpp. * gnu/packages/textutils.scm (utfcpp): New variable. --- g

Re: [PATCH 1/3] gnu: Add utfcpp.

2016-05-09 Thread Alex Kost
Leo Famulari (2016-05-09 06:31 +0300) wrote: > On Sun, May 08, 2016 at 09:43:28AM -0500, Alex Griffin wrote: >> On Sun, May 8, 2016, at 03:13 AM, Alex Kost wrote: >> > I don't know whether we have an idiomatic way to convert "2.3.4" into >> > "2_3_4", but I would just use a tarball from github: >>

Re: [PATCH 1/3] gnu: Add utfcpp.

2016-05-08 Thread Leo Famulari
On Sun, May 08, 2016 at 09:43:28AM -0500, Alex Griffin wrote: > On Sun, May 8, 2016, at 03:13 AM, Alex Kost wrote: > > I don't know whether we have an idiomatic way to convert "2.3.4" into > > "2_3_4", but I would just use a tarball from github: > > The project was just migrated to GitHub and hasn

Re: [PATCH 1/3] gnu: Add utfcpp.

2016-05-08 Thread Alex Kost
Alex Griffin (2016-05-08 17:43 +0300) wrote: > On Sun, May 8, 2016, at 03:13 AM, Alex Kost wrote: >> I suggest (gnu packages textutils); I see 'utf8proc' is placed there. I >> think this is also a suitable module for 'libunistring' (currently it is >> placed in its own file), but it's a separate

Re: [PATCH 1/3] gnu: Add utfcpp.

2016-05-08 Thread Alex Kost
Ludovic Courtès (2016-05-08 19:12 +0300) wrote: > Alex Kost skribis: > >> Not a big thing but we indent '#:use-module' by 2 spaces. Out of >> curiosity: what editor do you use? > > [...] > >> Please don't add leading spaces in descriptions :-) > > Just a side remark: I think it’s preferable and

Re: [PATCH 1/3] gnu: Add utfcpp.

2016-05-08 Thread Ludovic Courtès
Alex Kost skribis: > Not a big thing but we indent '#:use-module' by 2 spaces. Out of > curiosity: what editor do you use? [...] > Please don't add leading spaces in descriptions :-) Just a side remark: I think it’s preferable and more efficient if those of us with commit access fix little is

Re: [PATCH 1/3] gnu: Add utfcpp.

2016-05-08 Thread Alex Griffin
On Sun, May 8, 2016, at 03:13 AM, Alex Kost wrote: > I suggest (gnu packages textutils); I see 'utf8proc' is placed there. I > think this is also a suitable module for 'libunistring' (currently it is > placed in its own file), but it's a separate question. Okay. I had assumed textutils was for co

Re: [PATCH 1/3] gnu: Add utfcpp.

2016-05-08 Thread Alex Kost
Alex Griffin (2016-05-08 05:10 +0300) wrote: > On Sat, May 7, 2016, at 05:19 PM, Leo Famulari wrote: >> Not a bad idea for a new module, but I wonder if you considered any of >> the existing modules? Are none of them appropriate? > > Nothing else seemed to fit, but I'm open to suggestions. I sugg

Re: [PATCH 1/3] gnu: Add utfcpp.

2016-05-07 Thread Alex Griffin
On Sat, May 7, 2016, at 05:19 PM, Leo Famulari wrote: > Not a bad idea for a new module, but I wonder if you considered any of > the existing modules? Are none of them appropriate? Nothing else seemed to fit, but I'm open to suggestions. > > +(define-public utfcpp > > I'm curious — are there any

Re: [PATCH 1/3] gnu: Add utfcpp.

2016-05-07 Thread Leo Famulari
On Sat, May 07, 2016 at 01:36:21PM -0500, Alex Griffin wrote: > This is a dependency of ledger. It's bundled in ledger's source but I > think it's better to unbundle it. It's a bit unusual in that the headers > contain the whole source. > * gnu/packages/unicode.scm: New file. Not a bad idea for a