Skip to content

standardese 0.5.2#76423

Closed
alebcay wants to merge 2 commits intoHomebrew:masterfrom
alebcay:bump-standardese-0.5.2
Closed

standardese 0.5.2#76423
alebcay wants to merge 2 commits intoHomebrew:masterfrom
alebcay:bump-standardese-0.5.2

Conversation

@alebcay
Copy link
Copy Markdown
Member

@alebcay alebcay commented May 2, 2021

Created with brew bump-formula-pr.

@carlocab
Copy link
Copy Markdown
Member

carlocab commented May 2, 2021

Error: An exception occurred within a child process:
  Errno::ENOENT: No such file or directory - external/cppast/lib_cppast_tiny_process.dylib

Looks like they moved the stuff we used to install manually.

@alebcay
Copy link
Copy Markdown
Member Author

alebcay commented May 2, 2021

From what I can tell, this library is no longer built (maybe rolled into libcppast proper). It also looks like the cppast build process has moved from using Git submodules to using CMake to fetch content at build-time instead 🤦.

@carlocab
Copy link
Copy Markdown
Member

carlocab commented May 2, 2021

Doesn't surprise me.

In the last version bump, they changed their build so that standardese now relies on shared libraries that their build system doesn't install, so 🤷

@carlocab
Copy link
Copy Markdown
Member

carlocab commented May 2, 2021

In any case, I was planning on moving those libraries from libexec into lib so we can pass rpath to CMAKE_INSTALL_RPATH. (I put them in libexec last time since I wasn't really sure where to put stuff their build system doesn't install, but lib does seem like the right move here.)

@chenrui333 chenrui333 added the build failure CI fails while building the software label May 3, 2021
@github-actions
Copy link
Copy Markdown
Contributor

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@github-actions github-actions bot added the stale No recent activity label May 25, 2021
alebcay and others added 2 commits May 28, 2021 05:21
Also, use `assert_includes` rather than making a system call to `fgrep`.
@carlocab carlocab force-pushed the bump-standardese-0.5.2 branch from 3794984 to 7e173fd Compare May 28, 2021 04:52
@BrewTestBot BrewTestBot added the bump-formula-pr PR was created using `brew bump-formula-pr` label May 28, 2021
@carlocab carlocab removed build failure CI fails while building the software stale No recent activity labels May 28, 2021
@BrewTestBot
Copy link
Copy Markdown
Contributor

🤖 A scheduled task has triggered a merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bump-formula-pr PR was created using `brew bump-formula-pr`

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants