- **Comment**:
Now that the original change has been reverted, how should this ticket be
handled?
@hahnjo, would you like to move forward with your patch, remove the parts
related to `tidy` and adjust the issue description?
If you prefer, I could incorporate your changes (with credit) when I resubmit
my patch. I'm not sure what should be done with this ticket then—maybe set it
to Invalid?
---
** [issues:#5586] Fix 'check' target without tidy**
**Status:** Started
**Created:** Fri Oct 25, 2019 01:04 PM UTC by Jonas Hahnfeld
**Last Updated:** Sat Oct 26, 2019 08:52 AM 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 [email protected] 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
[email protected]
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
