It would probably cause less of a delay to combine the two in one issue (and 
most likely Dan's), but it might make sense to commit them as separate commits 
anyway (your aclocal change before Dan's the way it sounds: I haven't checked 
the actual code).


---

** [issues:#5586] Fix 'check' target without tidy**

**Status:** Started
**Created:** Fri Oct 25, 2019 01:04 PM UTC by Jonas Hahnfeld
**Last Updated:** Mon Oct 28, 2019 04:09 PM UTC
**Owner:** Jonas Hahnfeld


If configure didn't find tidy, it would assign an error message to the TIDY 
variable. Fix the logic of STEPMAKE_OPTIONAL_REQUIRED to leave a constant 
'false' in the variable if an optional program is not found. This can be easily 
checked and has the advantage that it is guaranteed to fail if executed.

http://codereview.appspot.com/573150043


---

Sent from sourceforge.net because testlilyissues-a...@lists.sourceforge.net is 
subscribed to https://sourceforge.net/p/testlilyissues/issues/

To unsubscribe from further messages, a project admin can change settings at 
https://sourceforge.net/p/testlilyissues/admin/issues/options.  Or, if this is 
a mailing list, you can unsubscribe from the mailing list.
_______________________________________________
Testlilyissues-auto mailing list
testlilyissues-a...@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
  • ... Auto mailings of changes to Lily Issues via Testlilyissues-auto via Automated messages for lilypond development
    • ... Auto mailings of changes to Lily Issues via Testlilyissues-auto via Automated messages for lilypond development
    • ... Auto mailings of changes to Lily Issues via Testlilyissues-auto via Automated messages for lilypond development

Reply via email to