search

mattn / p5-Devel-CheckLib

check that a library is available
http://search.cpan.org/dist/Devel-CheckLib/
13 stars 22 forks source link

Fix for dirs with space in #34

Closed mohawk2 closed 2 years ago

mohawk2 commented 3 years ago

The tests fail when the Perl executable path has a space in it. This fixes that.

mohawk2 commented 3 years ago

It also refactors the repetitive code that has annoyed me when I've had to debug my use of this excellent module, and adds a lot more debugging info when debug is true.

In this specific case, the problem I was having is that GCC was optimising away the test code I gave as function, so it linked fine, but the real code obviously then failed. This PR therefore also removes -O from $Config_ccflags.

mohawk2 commented 2 years ago

@mattn This would need your approval to run the CI.

mattn commented 2 years ago

I'll look this tonight.

mattn commented 2 years ago

Thank you. I'll ship this in later.