Re: [JMarc]: xforms/Makefile.am
Angus == Angus Leeming [EMAIL PROTECTED] writes: I am not really used to this {} syntax (I did not know it existed, actually). Angus Wow! You introduced it to me! (Remember some mails long ago Angus about ways to improve our shell-scripting?) In this particular Angus instance, the 'Useless use of 'test' idea is that we don't Angus need if test return status; then... fi but can use the return Angus status direct. I meant this use of {} for grouping commands (especially when the opening and closing brackets are not on the same line). Angus I shall leave that to you, if you don't mind. I'm reaching Angus auto-magic saturation point ;-) But the goal here is to remove some auto-magic :) JMarc
Re: [JMarc]: xforms/Makefile.am
> "Angus" == Angus Leeming <[EMAIL PROTECTED]> writes: >> I am not really used to this {} syntax (I did not know it existed, >> actually). Angus> Wow! You introduced it to me! (Remember some mails long ago Angus> about ways to improve our shell-scripting?) In this particular Angus> instance, the 'Useless use of 'test'" idea is that we don't Angus> need if test return status; then... fi but can use the return Angus> status direct. I meant this use of {} for grouping commands (especially when the opening and closing brackets are not on the same line). Angus> I shall leave that to you, if you don't mind. I'm reaching Angus> auto-magic saturation point ;-) But the goal here is to remove some auto-magic :) JMarc
Re: [JMarc]: xforms/Makefile.am
Jean-Marc Lasgouttes wrote: Angus == Angus Leeming [EMAIL PROTECTED] writes: Angus Jean-Marc, wouldn't this: stamp-forms: lyx_forms.h-tmp if cmp Angus -s $ lyx_forms.h ; then \ : ;\ else \ rm -f lyx_forms.h ;\ cp Angus $ lyx_forms.h ;\ fi echo timestamp $@ Angus be better as this: stamp-forms: lyx_forms.h-tmp cmp -s $ Angus lyx_forms.h || { \ rm -f lyx_forms.h ;\ cp $ lyx_forms.h ;\ Angus } Angus echo timestamp $@ I am not really used to this {} syntax (I did not know it existed, actually). Wow! You introduced it to me! (Remember some mails long ago about ways to improve our shell-scripting?) In this particular instance, the 'Useless use of 'test' idea is that we don't need if test return status; then... fi but can use the return status direct. But the right thing to do in this particular case is to get rid of these include files and go back to the good old #include FORMS_H_LOCATION we used to have. The reason why Lars introduced that (that rerunning configure with a different frontend forces a full rebuild) does not hold anymore now that we have multiple frontends support. I shall leave that to you, if you don't mind. I'm reaching auto-magic saturation point ;-) -- Angus
Re: [JMarc]: xforms/Makefile.am
Jean-Marc Lasgouttes wrote: >> "Angus" == Angus Leeming <[EMAIL PROTECTED]> writes: > > Angus> Jean-Marc, wouldn't this: stamp-forms: lyx_forms.h-tmp if cmp > Angus> -s $< lyx_forms.h ; then \ : ;\ else \ rm -f lyx_forms.h ;\ > cp Angus> $< lyx_forms.h ;\ fi echo timestamp > $@ > > Angus> be better as this: stamp-forms: lyx_forms.h-tmp cmp -s $< > Angus> lyx_forms.h || { \ rm -f lyx_forms.h ;\ cp $< lyx_forms.h ;\ > Angus> } > Angus> echo timestamp > $@ > > I am not really used to this {} syntax (I did not know it existed, > actually). Wow! You introduced it to me! (Remember some mails long ago about ways to improve our shell-scripting?) In this particular instance, the 'Useless use of 'test'" idea is that we don't need if test return status; then... fi but can use the return status direct. > But the right thing to do in this particular case is to get rid of > these include files and go back to the good old > #include FORMS_H_LOCATION > we used to have. The reason why Lars introduced that (that > rerunning configure with a different frontend forces a full rebuild) > does not hold anymore now that we have multiple frontends support. I shall leave that to you, if you don't mind. I'm reaching auto-magic saturation point ;-) -- Angus
Re: [JMarc]: xforms/Makefile.am
Angus == Angus Leeming [EMAIL PROTECTED] writes: Angus Jean-Marc, wouldn't this: stamp-forms: lyx_forms.h-tmp if cmp Angus -s $ lyx_forms.h ; then \ : ;\ else \ rm -f lyx_forms.h ;\ cp Angus $ lyx_forms.h ;\ fi echo timestamp $@ Angus be better as this: stamp-forms: lyx_forms.h-tmp cmp -s $ Angus lyx_forms.h || { \ rm -f lyx_forms.h ;\ cp $ lyx_forms.h ;\ Angus } Angus echo timestamp $@ I am not really used to this {} syntax (I did not know it existed, actually). But the right thing to do in this particular case is to get rid of these include files and go back to the good old #include FORMS_H_LOCATION we used to have. The reason why Lars introduced that (that rerunning configure with a different frontend forces a full rebuild) does not hold anymore now that we have multiple frontends support. JMarc
Re: [JMarc]: xforms/Makefile.am
> "Angus" == Angus Leeming <[EMAIL PROTECTED]> writes: Angus> Jean-Marc, wouldn't this: stamp-forms: lyx_forms.h-tmp if cmp Angus> -s $< lyx_forms.h ; then \ : ;\ else \ rm -f lyx_forms.h ;\ cp Angus> $< lyx_forms.h ;\ fi echo timestamp > $@ Angus> be better as this: stamp-forms: lyx_forms.h-tmp cmp -s $< Angus> lyx_forms.h || { \ rm -f lyx_forms.h ;\ cp $< lyx_forms.h ;\ Angus> } Angus> echo timestamp > $@ I am not really used to this {} syntax (I did not know it existed, actually). But the right thing to do in this particular case is to get rid of these include files and go back to the good old #include FORMS_H_LOCATION we used to have. The reason why Lars introduced that (that rerunning configure with a different frontend forces a full rebuild) does not hold anymore now that we have multiple frontends support. JMarc