mirror of
https://github.com/ruby/ruby.git
synced 2025-08-15 13:39:04 +02:00
Try removing building C API specs in CRuby makefiles entirely
* Since it does not work when using some configure options such as: .../configure --with-ext=-test-/cxxanyargs,+ --enable-shared as the CI does. * It also duplicates the logic of spec/ruby/optional/capi/spec_helper.rb incorrectly. ruby/spec maintainers have no experience and no interest in dealing with these complicated CRuby build system issues. We asked help on the CRuby Slack and nobody helped so far.
This commit is contained in:
parent
29be534fad
commit
2a9236366d
Notes:
git
2025-05-09 21:22:30 +00:00
2 changed files with 0 additions and 52 deletions
|
@ -504,26 +504,6 @@ update-deps:
|
|||
$(GIT) --git-dir=$(GIT_DIR) merge --no-edit --ff-only $(update_deps)
|
||||
$(GIT) --git-dir=$(GIT_DIR) branch --delete $(update_deps)
|
||||
|
||||
# order-only-prerequisites doesn't work for $(RUBYSPEC_CAPIEXT)
|
||||
# because the same named directory exists in the source tree.
|
||||
$(RUBYSPEC_CAPIEXT)/%.$(DLEXT): $(srcdir)/$(RUBYSPEC_CAPIEXT)/%.c $(srcdir)/$(RUBYSPEC_CAPIEXT)/rubyspec.h $(RUBY_H_INCLUDES) $(LIBRUBY)
|
||||
$(ECHO) building $@
|
||||
$(Q) $(MAKEDIRS) $(@D)
|
||||
$(Q) $(DLDSHARED) -L. $(XDLDFLAGS) $(XLDFLAGS) $(LDFLAGS) $(INCFLAGS) -I$(EXTOUT)/include $(CPPFLAGS) $(OUTFLAG)$@ $< $(LIBRUBYARG)
|
||||
ifneq ($(POSTLINK),)
|
||||
$(Q) $(POSTLINK)
|
||||
endif
|
||||
$(Q) $(RMALL) $@.*
|
||||
|
||||
RUBYSPEC_CAPIEXT_SO := $(patsubst %.c,$(RUBYSPEC_CAPIEXT)/%.$(DLEXT),$(notdir $(wildcard $(srcdir)/$(RUBYSPEC_CAPIEXT)/*.c)))
|
||||
rubyspec-capiext: $(RUBYSPEC_CAPIEXT_SO)
|
||||
@ $(NULLCMD)
|
||||
|
||||
ifeq ($(ENABLE_SHARED),yes)
|
||||
ruby: $(if $(LIBRUBY_SO_UPDATE),$(RUBYSPEC_CAPIEXT_SO))
|
||||
exts: rubyspec-capiext
|
||||
endif
|
||||
|
||||
spec/%/ spec/%_spec.rb: programs exts PHONY
|
||||
+$(RUNRUBY) -r./$(arch)-fake $(srcdir)/spec/mspec/bin/mspec-run -B $(srcdir)/spec/default.mspec $(SPECOPTS) $(patsubst %,$(srcdir)/%,$@)
|
||||
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue