Skip to content

Make "make exts" build only gems listed in gems/bundled_gems #7119

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

Closed
wants to merge 2 commits into from

Conversation

mame
Copy link
Member

@mame mame commented Jan 13, 2023

template/exts.mk.tmpl includes older directories such as debug-1.5.0, which leads to a failure of "make".

make[2]: ディレクトリ '/home/mame/work/ruby/.bundle/gems/debug-1.5.0/ext/debug' に入ります
make[2]: *** '../../../../../.bundle/specifications/debug-1.5.0.gemspec' に必要なターゲット '../../../../.././.bundle/gems/debug-1.5.0/.bundled.debug-1.5.0.gemspec' を make するルールがありません.  中止.

We should ignore older source directories in .bundle/gems. If you want to remove them, use "make outdate-bundled-gems".

template/exts.mk.tmpl includes older directories such as debug-1.5.0,
which leads to a failure of "make".

```
make[2]: ディレクトリ '/home/mame/work/ruby/.bundle/gems/debug-1.5.0/ext/debug' に入ります
make[2]: *** '../../../../../.bundle/specifications/debug-1.5.0.gemspec' に必要なターゲット '../../../../.././.bundle/gems/debug-1.5.0/.bundled.debug-1.5.0.gemspec' を make するルールがありません.  中止.
```

We should ignore older source directories in .bundle/gems.
If you want to remove them, use "make outdate-bundled-gems".
@mame mame force-pushed the exts-only-for-gems-bundled_gems branch from 7815004 to a826a8a Compare January 16, 2023 02:08
@mame mame force-pushed the exts-only-for-gems-bundled_gems branch from a826a8a to 0a9a3aa Compare January 17, 2023 08:10
@mame
Copy link
Member Author

mame commented Jan 26, 2023

This is no longer needed thanks to #7146

@mame mame closed this Jan 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant