Skip to content

Commit

Permalink
Makefile.inc: s/curl_SOURCES/CURL_FILES
Browse files Browse the repository at this point in the history
This allows the root Makefile.am to include the Makefile.inc without
causing automake to warn on it (variables named *_SOURCES are
magic). curl_SOURCES is then instead assigned properly in
src/Makefile.am only.

Closes curl#577
  • Loading branch information
bagder committed Dec 23, 2015
1 parent a464f33 commit bfe6f1f
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 3 deletions.
3 changes: 2 additions & 1 deletion src/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -43,9 +43,10 @@ if(MSVC)
list(APPEND CURL_SOURCE curl.rc)
endif()

# CURL_FILES comes from Makefile.inc
add_executable(
${EXE_NAME}
${curl_SOURCES}
${CURL_FILES}
)

source_group("cURLX source files" FILES ${CURLX_CFILES})
Expand Down
3 changes: 3 additions & 0 deletions src/Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,9 @@ endif

include Makefile.inc

# CURL_FILES comes from Makefile.inc
curl_SOURCES = $(CURL_FILES)

# This might hold -Werror
CFLAGS += @CURL_CFLAG_EXTRAS@

Expand Down
4 changes: 2 additions & 2 deletions src/Makefile.inc
Original file line number Diff line number Diff line change
Expand Up @@ -111,5 +111,5 @@ CURL_HFILES = \

CURL_RCFILES = curl.rc

curl_SOURCES = $(CURL_CFILES) $(CURLX_CFILES) $(CURL_HFILES)

# curl_SOURCES is special and gets assigned in src/Makefile.am
CURL_FILES = $(CURL_CFILES) $(CURLX_CFILES) $(CURL_HFILES)

0 comments on commit bfe6f1f

Please sign in to comment.