This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Remove symlinks created in argv0-symlink.exp and general cleanup


On Wed, Aug 12, 2015 at 11:43 AM, Joel Brobecker <brobecker@adacore.com> wrote:
>> I suspect, though I haven't tried, if we remove the
>> check-parallel/check-single test, it would be straightforward to
>> remove the GNU make requirement and just move the details to a wrapper
>> script.
>
> Dependency tracking also requires GNU/Make; would you be able to
> take care of that through the wrapper?

Maybe I missed something but I didn't see any real dependency
tracking as far as the invocation of runtest goes.
Clearly the current implementation uses dependencies
to implement check-parallel, but I wouldn't call those
dependencies per-se.

>> Always invoking runtest by a wrapper script has other benefits.
>>
>> I'm not advocating for this, I only ever use GNU make.
>> But if the topic comes up, it's not clear to me GNU make
>> is an absolute requirement.
>
> It would be nice to list the additional benefits and decide whether
> we want to go that route or not, ideally more-or-less independently
> of whether we want to require GNU/Make or not.
>
> The reason why I am advocating in favor of requiring GNU/Make is that
> we are regularly hindered or doing extra work trying to support non-
> GNU/Make, while on the other hand, the community affected by this
> proposal only need to update their build procedures to put GNU/Make
> on their PATH. And since GCC already requires GNU/Make, I cannot
> understand why it would be difficult to do so.

How often does it come up in the context of "make check"
(beyond the check-parallel implementation).
Other reasons for wanting to require GNU/Make are outside the scope
of my email. :-)


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]