search

scheme / scsh

A Unix shell embedded in scheme
Other
386 stars 36 forks source link

Don't not include util.h if libutil.h is available. #49

Closed mikesperber closed 8 months ago

mikesperber commented 8 months ago

macOS has openpty in util.h, but also has libutil.h.

roderyc commented 8 months ago

Hey Mike! Sorry, I'm confused by this change - isn't the point of break to do exactly what you want in the commit message? The way I'm reading the doc, it should break out of the loop of checking each lib once it finds one of them available, given the break keyword. If anything it seems this change would include both on systems where both are available.

For what it's worth, on my macOS system this change has no effect on the autoconf output - libutil.h can't be found either way.

Edit: Oop, misread the double negative 😅. Makes more sense now.

mikesperber commented 8 months ago

@roderyc Thanks!