mbox series

[bug#33540,v2,0/3] Add lttoolbox, apertium

Message ID 20181203165024.25334-1-arunisaac@systemreboot.net
Headers show
Series Add lttoolbox, apertium | expand

Message

Arun Isaac Dec. 3, 2018, 4:50 p.m. UTC 
I am using the release tarballs for both lttoolbox and apertium, since they
are the asset tarballs and not autogenerated ones. Also, it turns out that the
hash change of the lttoolbox tarball was due to a configure script being
added. Therefore the patch-autogen phase is no longer necessary.

Like you mentioned, apertium does have a submodule for apertium-get. But, we
cannot use it because we are using the release tarball.

The other suggested changes are not a problem. I have made them. The updated
patches follow. If there's nothing else, I'll push to master in 24 hours.

Arun Isaac (3):
  gnu: dictionaries: Use license prefix.
  gnu: Add lttoolbox.
  gnu: Add apertium.

 gnu/packages/dictionaries.scm | 107 +++++++++++++++++++++++++++++++---
 1 file changed, 99 insertions(+), 8 deletions(-)

Comments

Ludovic Courtès Dec. 4, 2018, 8:36 a.m. UTC | #1 
Arun Isaac <arunisaac@systemreboot.net> skribis:

> I am using the release tarballs for both lttoolbox and apertium, since they
> are the asset tarballs and not autogenerated ones. Also, it turns out that the
> hash change of the lttoolbox tarball was due to a configure script being
> added. Therefore the patch-autogen phase is no longer necessary.
>
> Like you mentioned, apertium does have a submodule for apertium-get. But, we
> cannot use it because we are using the release tarball.
>
> The other suggested changes are not a problem. I have made them. The updated
> patches follow. If there's nothing else, I'll push to master in 24 hours.

Alright!

Thanks,
Ludo’.
Arun Isaac Dec. 4, 2018, 6:41 p.m. UTC | #2 
Pushed to master. Thanks for the review!