On Mon, Nov 27, 2017 at 5:15 PM, Michael Orlitzky <[email protected]> wrote:
> On 11/27/2017 09:00 AM, Erik Bray wrote:
>>
>>> On the other hand, if we standardize on the name of the m4 file (i.e.
>>> it's fixed but optional), then you could use "sinclude" to grab them all
>>> at once, ignoring the errors from the ones that don't exist.
>>
>> I'm not really sure what you mean here.
>
> Nothing ground-breaking, only that you might do something like...
>
>   for d in build/pkgs/*; do
>     echo "sinclude(${d}/spkg-configure.m4)"
>   done
>
> without having to care whether or not spkg-configure.m4 exists in each
> directory.
>
> Now I'm curious why PHP does the loop in the shell script... how does
> your "m4_foreach_w" loop behave if "find" outputs a path containing a space?

In this case it won't unless there's a mistake in the source directory
tree (it should only return names like
"build/pkgs/<pkgname>/spkg-configure.m4", and package names can't
contain spaces.

If we wanted to make it more robust though that's easy enough.

-- 
You received this message because you are subscribed to the Google Groups 
"sage-devel" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at https://groups.google.com/group/sage-devel.
For more options, visit https://groups.google.com/d/optout.

Reply via email to