You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@daffodil.apache.org by GitBox <gi...@apache.org> on 2022/11/10 14:59:10 UTC

[GitHub] [daffodil] stevedlawrence commented on pull request #872: Add macOS build support

stevedlawrence commented on PR #872:
URL: https://github.com/apache/daffodil/pull/872#issuecomment-1310420953

   > The Build Documentation step generates 1 warning
   
   There's lots of discussions on the internet about this error, with I think no real resolution. I think this might be a known issue. 
   
   > The Run Integration Tests step now generates 3 warnings;
   
   This was a change made in the recent integration test refactoring. We used to just have the blob tests commented out because they require manually generating large files to test. But because they were commented out, they were easy to forget about. To avoid that, I added some JUnit `assumeTrue` calls so that the tests would be skipped if those generated files didn't exist. That way the tests still pass, but we get a clear reminder that those tests exist and were skipped. We don't need to run these very often because blob logic doesn't change much, but they're good to know have be periodically reminded of.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@daffodil.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org