-
Notifications
You must be signed in to change notification settings - Fork 373
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Various minor updates, v4 metadata changes. #973
Conversation
</Filter> | ||
<Filter Include="src\.libs"> | ||
<UniqueIdentifier>{B726DF7D-6D1D-48FB-0000-000000000002}</UniqueIdentifier> | ||
</Filter> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pmienk Could you update the build system to prevent this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will look at it now. Looks like you'd like to exclude anything 'hidden' - from both file and directory listings?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are a few hidden files that we use, but no hidden directories. Not sure if the hidden files are relevant to the source listings though.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I had some errand .gitignore show up in one file listing, so I'm hoping we can ignore all of them or I can track it down and remove it. Its contents can move into the top-level one. But I'll rewrite the discovery function to allow for directory and file ignoring as separate parameters.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Addressed by libbitcoin/libbitcoin-build#164. I did not see any use of hidden files and so have currently excluded them. Also finally pulled the file discovery code into utilities (it's also used in my in-progress cmake build)
Please do not merge until corresponding changes in other repos are ready.