[Lame-dev] [ lame-Bugs-1776943 ] No lame.exe or DLL

[SourceForge.net <no...@so...>]

Bugs item #1776943, was opened at 2007-08-18 11:15
Message generated for change (Comment added) made by nobody
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=100290&aid=1776943&group_id=290

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: configure/Makefiles
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Nobody/Anonymous (nobody)
Assigned to: Nobody/Anonymous (nobody)
Summary: No lame.exe or DLL

Initial Comment:
I am using MSVC 6 on Windows XP with the Intel 6 compiler.

Using the new makefile.MSVC 1.70 with the following command line yields only one file in .\output (libmp3lame-static.lib). No lame.exe or DLL is produced at all. No error messages or warnings appear during compilation.

nmake -f makefile.msvc COMP=INTEL ASM=YES MMX=YES SNDFILE=NO

----------------------------------------------------------------------

Comment By: Nobody/Anonymous (nobody)
Date: 2007-08-22 07:34

Message:
Logged In: NO 

Thanks for the makefile fix. The only issue remaining is that the DLL
isn't built by default, but that's easy enough to fix.

----------------------------------------------------------------------

Comment By: Nobody/Anonymous (nobody)
Date: 2007-08-18 12:09

Message:
Logged In: NO 

MORE INFORMATION

/opt:NOWIN98 is a VC only option. It confuses the Intel compiler. It only
needs to be removed from line 340.

----------------------------------------------------------------------

Comment By: Nobody/Anonymous (nobody)
Date: 2007-08-18 11:53

Message:
Logged In: NO 

The problems appear to be as follows:

1) Line 52 is missing $(T_DLL)
2) The /opt:NOWIN98 option replaces all of the other options previously
set ... nothing gets built.
3) The $(LN_OUT)$(TARGET_DIR)$@ results in errors like "Cannot open
.\output.\output\lame.exe"

----------------------------------------------------------------------

Comment By: Nobody/Anonymous (nobody)
Date: 2007-08-18 11:51

Message:
Logged In: NO 

The following patch fixes my problems:

Index: Makefile.MSVC
===================================================================
RCS file: /cvsroot/lame/lame/Makefile.MSVC,v
retrieving revision 1.70
diff -u -r1.70 Makefile.MSVC
--- Makefile.MSVC	12 Aug 2007 18:30:31 -0000	1.70
+++ Makefile.MSVC	18 Aug 2007 18:41:31 -0000
@@ -49,7 +49,7 @@
 
 #	default targets
 
-PGM = $(T_LAME)
+PGM = $(T_LAME) $(T_DLL)
 
 #	some default settings
 
@@ -233,8 +233,8 @@
 
 CC_OPTS = $(CC_OPTS) /DNDEBUG /MT
 
-LN_OPTS = /nologo /opt:NOWIN98 /pdb:none
-LN_DLL = /nologo /DLL /opt:NOWIN98
+LN_OPTS = /nologo /pdb:none
+LN_DLL = /nologo /DLL
 
 CC_OUT = /Fo
 LN_OUT = /force /OUT:
@@ -337,7 +337,7 @@
           /Qunroll /Qsox- /Qip $(CC_OPTS)
 
 
-LN_OPTS = $(CC_OPTS) /opt:NOWIN98
+LN_OPTS = $(CC_OPTS)
 LN_DLL = /LD
 LN_OUT = /Fe$(TARGET_DIR)
 CC_OUT = /Fo
@@ -601,21 +601,21 @@
 lib: $(ASM_OBJ) libA libB
 
 $(T_LAME) : target_directory config.h $(T_LIB_STATIC) common $(LAME_OBJ)
-	@$(LN) $(LN_OUT)$(TARGET_DIR)$@ $(LN_SWITCHES) $(LIBSNDFILE) \
+	@$(LN) $(LN_OUT)$@ $(LN_SWITCHES) $(LIBSNDFILE) \
 	       $(TARGET_DIR)$(T_LIB_STATIC) $(CMMN_OBJ) $(LAME_OBJ) $(ADDL_OBJ)
 	@echo.
 	@echo --=*  $(TARGET_DIR)$@ ready  *=--
 	@echo.
 
 $(T_MP3X) : target_directory config.h lib common $(MPX_OBJ)
-	@$(LN) $(LN_OUT)$(TARGET_DIR)$@ $(LN_SWITCHES) $(GTK_LIBS) $(LIBSNDFILE)
\
+	@$(LN) $(LN_OUT)$@ $(LN_SWITCHES) $(GTK_LIBS) $(LIBSNDFILE) \
 	       $(TARGET_DIR)$(T_LIB_STATIC) $(CMMN_OBJ) $(MPX_OBJ) $(ADDL_OBJ)
 	@echo.
 	@echo --=*  $(TARGET_DIR)$@ ready  *=--
 	@echo.
 
 $(T_LEGACY_DLL) : target_directory config.h $(DLL_OBJ)
-	@$(LN) $(LN_OUT)$(TARGET_DIR)$@ $(LN_SWITCHES) \
+	@$(LN) $(LN_OUT)$@ $(LN_SWITCHES) \
 	       $(TARGET_DIR)$(T_LIB_STATIC) $(LN_DLL) \
 	       $(DLL_OBJ) $(ADDL_OBJ)
 	@echo.
@@ -624,9 +624,9 @@
 
 $(T_DLL) : target_directory config.h $(T_LIB_STATIC) 
 	@$(LN) $(LN_DLL) $(MACHINE) \
-		/def:"include\lame.def" \
+		/DEF:"include\lame.def" \
 		$(ADDL_OBJ) \
-		$(LN_OUT)"$(TARGET_DIR)$@" \
+		$(LN_OUT)$@ \
 		$(TARGET_DIR)$(T_LIB_STATIC) libmp3lame\version.obj 
 	@echo.
 	@echo --=*  $(TARGET_DIR)$@ ready  *=--
@@ -634,7 +634,7 @@
 
 $(T_LIB_STATIC) : target_directory lib
 	@lib $(LIB_OPTS) \
-		/out:"$(TARGET_DIR)$@" \
+		/out:$(TARGET_DIR)$@ \
 		$(ASM_OBJ) $(LIB_OBJ) $(MPG_OBJ)
 	@echo.
 	@echo --=*  $(TARGET_DIR)$@ ready  *=--


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=100290&aid=1776943&group_id=290