Re: [PATCH] gnu: add vte-ng

[Mckinley Olsen]

Hey Alex, I'm hoping this patch is up to snuff.
I've changed the package definition to depend on the latest tagged
source and provided a synopsis & description specific to vte-ng.
Let me know if you have any other concerns that need to be addressed.
Thanks!

On Sun, Apr 10, 2016 at 1:42 AM, Alex Kost <address@hidden> wrote:
> Mckinley Olsen (2016-04-09 21:10 +0300) wrote:
>
>> Sorry, I'm new to mailing lists; hope this gets to the right place
>
> Oops, it wasn't Cc-ed to guix-devel list, but since I'm replying, the
> others will see it anyway.
>
>> I used the git checkout, as I started writing termite's package first
>> and realized it needed a recursive checkout; of course I copied over
>> that implementation to here. Thanks for the suggestion to use the
>> tarball, I've made that change
>>
>> I used the version I did because I was using the package definition
>> in Nix as a guideline. I've worked through packaging a slightly more
>> up-to-date version (submitted patch to update pcre2 to required
>> version), but am stuck on getting versions 0.43.91 and newer packaged
>> due to an error (vteconv.cc:774:40: error: missing sentinel in
>> function call [-Werror=format=]), which I believe was caused by this
>> change: https://bugzilla.gnome.org/show_bug.cgi?id=762863
>>
>> I'm quite new to the gnu build system, so it has taken me some time
>>
>> Sorry I haven't been able to get the newest packaged up. I've just
>> found that the gnome-updates branch contains some code working
>> through the exact error I'm seeing with vte-ng, so I might reach out
>> for them or wait for it to be merged.
>
> Ah, I think it's ok to wait for gnome-updates merge.  Thanks for working
> on the new packages!
>
> --
> Alex

0001-gnu-packages-gnome.scm-vte-ng-New-variable.patch
Description: Binary data