Skip to content

Commit

Permalink
depend: fix for static library on mswin
Browse files Browse the repository at this point in the history
* enc/depend (ARFLAGS): VisualC++ linker does not allow spaces between
  output option and the output file name.  [Bug ruby#7950]
* enc/depend (RANLIB): set default command to do nothing, or make the
  entire line a label on Windows.

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@39489 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
  • Loading branch information
nobu committed Feb 25, 2013
1 parent 593ffbc commit 72aee03
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 2 deletions.
8 changes: 8 additions & 0 deletions ChangeLog
Original file line number Diff line number Diff line change
@@ -1,3 +1,11 @@
Mon Feb 25 15:47:18 2013 Nobuyoshi Nakada <nobu@ruby-lang.org>

* enc/depend (ARFLAGS): VisualC++ linker does not allow spaces between
output option and the output file name. [Bug #7950]

* enc/depend (RANLIB): set default command to do nothing, or make the
entire line a label on Windows.

Mon Feb 25 14:41:07 2013 Nobuyoshi Nakada <nobu@ruby-lang.org>

* lib/mkmf.rb (MakeMakefile#init_mkmf): default libdirname to libdir.
Expand Down
1 change: 1 addition & 0 deletions enc/Makefile.in
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,7 @@ LIBRUBYARG_STATIC = $(LIBRUBYARG_SHARED)
empty =
AR = @AR@
CC = @CC@
ARFLAGS = @ARFLAGS@$(empty)
RANLIB = @RANLIB@
OUTFLAG = @OUTFLAG@$(empty)
COUTFLAG = @COUTFLAG@$(empty)
Expand Down
6 changes: 4 additions & 2 deletions enc/depend
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,8 @@
% rule_subst = CONFIG["RULE_SUBST"] || "%s"
% transvpath = rule_subst.dup.sub!(/\{[^{}]+\}/, '$(TRANSVPATH)/') || "enc/trans/%s"
% transvpath_prefix = (rule_subst.dup.sub!(/\{[^{}]+\}/, '{$(TRANSVPATH)}') || "%s") % ""
% CONFIG['ARFLAGS'] = 'rcu ' if CONFIG['ARFLAGS'].empty?
% CONFIG['RANLIB'] = ':' if CONFIG['RANLIB'].empty?
% if File::ALT_SEPARATOR
% pathrep = proc {|path| path.gsub('/', File::ALT_SEPARATOR).gsub(/\$\(([@<?*]\w?|\w+)\)/, "$(\\1:/=\\#{File::ALT_SEPARATOR})")}
% else
Expand Down Expand Up @@ -63,12 +65,12 @@ libtrans: $(LIBTRANS)
$(LIBENC): $(ENCOBJS)
@$(RM) $@
$(ECHO) linking statically-linked encoding library $@
$(Q) $(AR) <%=CONFIG['ARFLAGS'] || 'rcu' %> $@ $(ENCOBJS)
$(Q) $(AR) $(ARFLAGS)$@ $(ENCOBJS)
@-$(RANLIB) $@ 2> /dev/null || true
$(LIBTRANS): $(TRANSOBJS)
@$(RM) $@
$(ECHO) linking statically-linked transcoder library $@
$(Q) $(AR) <%=CONFIG['ARFLAGS'] || 'rcu' %> $@ $(TRANSOBJS)
$(Q) $(AR) $(ARFLAGS)$@ $(TRANSOBJS)
@-$(RANLIB) $@ 2> /dev/null || true

srcs: $(TRANSCSRCS)
Expand Down

0 comments on commit 72aee03

Please sign in to comment.