macports-base
macports-base copied to clipboard
Abort 'port test' early if test.run is set to no
This will avoid installing dependencies if tests is not enabled.
Closes: https://trac.macports.org/ticket/45010
Hm, it seems that this actually breaks a couple of tests. Please take a look.
Yea I missed that.... strange a lot of tests under tests/test fail even though they don't set test.run.
When running those tests, the target is "test" and they don't have test.run in the Portfiles so it fails. I wonder since the tests Portfiles all have 'test {}' in them, that this is why these fail.
I'm not sure this is the best or even correct way to fix this.
Before I work on this again, does anyone object or have suggesting for a better way to handle the test portion? It does work as-is, just seems hackish.
Just getting around to reviewing these PRs, so my apologies for not responding sooner.
I'll try to circle back on this over the coming weeks, when I can take a breather.
Regardless, I like the idea!
Shouldn't be needed any more after ddc2655.