Re: [bug-gawk] Suggestion on pc/Makefile

[Vincent Belaïche]

Answers below...

Le 26/10/2016 à 16:25, Eli Zaretskii a écrit :
>> From: address@hidden (Vincent Belaïche)
>> Date: Wed, 26 Oct 2016 15:42:30 +0200
>> Cc: Vincent Belaïche <address@hidden>
>>
>> Attached is a patch to pc/Makefile that allows to build without prior
>> copying of source files to the pc/ directory.
>
> That's not what you should do, according to README: you should copy
> the files _from_ pc/ to its parent directory instead.
>

Ooops... done the other way round, I should have read more carefully
README.pc.

>> I found also that in the orignal Makefile the 'pc/gawkmisc.pc' had to be
>> replaced by 'gawkmisc.pc'.
>
> Not if you build according to instructions.
>
>> Not sure whether you like it, it might be too GNU-make dependent.
>
> Yeah, I don't really see a reason to change that.

A reason is that you don't need to move files around with the modified
Makefile (OK, then for this contribution to be complete I should also
modify the Makefile.ext). Well, this is not a big deal to move them, but
it would make the README.pc simpler to understand for exhausted brains
like mine if you just write something like that:

--8<----8<----8<----8<----8<-- begin -->8---->8---->8---->8---->8----
  do:

     cd pc
     make

  then select a TARGET from those listed and do:

    make TARGET
--8<----8<----8<----8<----8<--  end  -->8---->8---->8---->8---->8----


> The existing Makefile works well, did so for many years.

Well, sorry for the noise & disturbance, I agree that I was too quick in
writing this email.

At least the 'awkgram.c' is to be added to 'clean' target. Hopefully,
this point makes my email not a complete disturbance.

Maybe you prefer to have a 'realclean' and a 'clean' target, and
'awkgram.c' erased only under 'realclean'.

VBR,
        Vincent.

---
L'absence de virus dans ce courrier électronique a été vérifiée par le logiciel 
antivirus Avast.
https://www.avast.com/antivirus