Re: [voidlinux/void-packages] chromium: update to 54.0.2840.100. (#5182)

Michael Gehring at Wed, 16 Nov 2016 22:45:51 -0800
ebfe commented on this pull request. > @@ -1,14 +1,3 @@ ---- ./third_party/usrsctp/usrsctplib/usrsctplib/netinet/sctp_userspace.c.orig -+++ ./third_party/usrsctp/usrsctplib/usrsctplib/netinet/sctp_userspace.c -@@ -72,7 +72,7 @@ - pthread_setname_np(name); - #endif - #if defined(__Userspace_os_Linux) -- pthread_setname_np(pthread_self(), name); -+ //pthread_setname_np(pthread_self(), name); I think this is available on musl with -D_GNU_SOURCE.
Michael Gehring at Wed, 16 Nov 2016 23:13:19 -0800
ebfe commented on this pull request. > @@ -1,14 +1,3 @@ ---- ./third_party/usrsctp/usrsctplib/usrsctplib/netinet/sctp_userspace.c.orig -+++ ./third_party/usrsctp/usrsctplib/usrsctplib/netinet/sctp_userspace.c -@@ -72,7 +72,7 @@ - pthread_setname_np(name); - #endif - #if defined(__Userspace_os_Linux) -- pthread_setname_np(pthread_self(), name); -+ //pthread_setname_np(pthread_self(), name); Euh, too early...
Christian Neukirchen at Thu, 17 Nov 2016 04:42:05 -0800
Just cosmetics, but I think `conf+='foo="bar"'` is nicer than quoting `\"`.
Christian Neukirchen at Thu, 17 Nov 2016 04:46:39 -0800
Drop `# XXX: trigger rebuild`. Also, I feel like the gn configuring and pre-ninja part should be in `do_configure`.
Enno Boland at Fri, 25 Nov 2016 04:08:28 -0800
ready to merge?