Re: sort of new branch: next

[Jim Meyering]

Pádraig Brady <address@hidden> wrote:
> Jim Meyering wrote:
>> I've pushed a few changes to a new "next" branch.
>> I expect to rebase it against master.
>>
>>   $ g shortlog HEAD ^master
>>         build: use dist-xz, not dist-lzma
>>         cleanup/modernize: don't test HAVE_MBRTOWC; now gnulib provides it
>>         portability: accommodate gnulib's getaddrinfo change
>>         maint: move coreutils-specific syntax check rules into cfg.mk
>>         maint: tighten m4 AC_ quoting check
>>         maint: use more sc_-prefixed names
>>         maint: factor out common code; prohibit use of HAVE_MBRTOWC
>
> The above look sensible from a _very_ quick review

Thanks for looking.

>>         bootstrap: don't use cmp's -s option when reading from a pipe
>
> I see cmp used all over the place (provided by diffutils on my fedora box).
> Should I be adding cmp to the requirements in bootstrap.conf?

No need, since cmp is one of those tools that's been assumed
to be present since time immemorial.

> I also noted that you can restrict the following function
> to just coreutils apps as follows?
>
> insert_sorted_if_absent() {
>   file=$1
>   str=$2
>   test -f $file || touch $file
>   test "$((echo "$str"; uniq $file) | LC_ALL=C sort | uniq -d)" ||
>   echo "$str" | sort -u - $file -o $file ||
>   exit 1
> }

Thanks, but it's not worth it, since you can depend on cmp.