]> git.djapps.eu Git - pkg/ggml/sources/llama.cpp/commitdiff
Name Migration: Build the deprecation-warning 'main' binary every time (#8404)
authorClint Herron <redacted>
Wed, 10 Jul 2024 16:35:18 +0000 (12:35 -0400)
committerGitHub <redacted>
Wed, 10 Jul 2024 16:35:18 +0000 (12:35 -0400)
* Modify the deprecation-warning 'main' binary to build every time, instead of only when a legacy binary is present. This is to help users of tutorials and other instruction sets from knowing what to do when the 'main' binary is missing and they are trying to follow instructions.

* Adjusting 'server' name-deprecation binary to build all the time, similar to the 'main' legacy name binary.

Makefile

index 68197fef80019910ae9b54aa1a54f0e6b5c74a59..668b38b99c3129c16c3df02c6adcc14ccaf5324c 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -1513,15 +1513,17 @@ llama-q8dot: pocs/vdot/q8dot.cpp ggml/src/ggml.o \
 # Mark legacy binary targets as .PHONY so that they are always checked.
 .PHONY: main quantize perplexity embedding server finetune
 
+# NOTE: We currently will always build the deprecation-warning `main` and `server` binaries to help users migrate.
+#  Eventually we will want to remove these target from building all the time.
 main: examples/deprecation-warning/deprecation-warning.cpp
-ifneq (,$(wildcard main))
        $(CXX) $(CXXFLAGS) -c $< -o $(call GET_OBJ_FILE, $<)
        $(CXX) $(CXXFLAGS) $(filter-out $<,$^) $(call GET_OBJ_FILE, $<) -o $@ $(LDFLAGS)
-       @echo "#########"
-       @echo "WARNING: The 'main' binary is deprecated. Please use 'llama-cli' instead."
-       @echo "  Remove the 'main' binary to remove this warning."
-       @echo "#########"
-endif
+       @echo "NOTICE: The 'main' binary is deprecated. Please use 'llama-cli' instead."
+
+server: examples/deprecation-warning/deprecation-warning.cpp
+       $(CXX) $(CXXFLAGS) -c $< -o $(call GET_OBJ_FILE, $<)
+       $(CXX) $(CXXFLAGS) $(filter-out %.h $<,$^) $(call GET_OBJ_FILE, $<) -o $@ $(LDFLAGS)
+       @echo "NOTICE: The 'server' binary is deprecated. Please use 'llama-server' instead."
 
 quantize: examples/deprecation-warning/deprecation-warning.cpp
 ifneq (,$(wildcard quantize))
@@ -1553,16 +1555,6 @@ ifneq (,$(wildcard embedding))
        @echo "#########"
 endif
 
-server: examples/deprecation-warning/deprecation-warning.cpp
-ifneq (,$(wildcard server))
-       $(CXX) $(CXXFLAGS) -c $< -o $(call GET_OBJ_FILE, $<)
-       $(CXX) $(CXXFLAGS) $(filter-out %.h $<,$^) $(call GET_OBJ_FILE, $<) -o $@ $(LDFLAGS)
-       @echo "#########"
-       @echo "WARNING: The 'server' binary is deprecated. Please use 'llama-server' instead."
-       @echo "  Remove the 'server' binary to remove this warning."
-       @echo "#########"
-endif
-
 finetune: examples/deprecation-warning/deprecation-warning.cpp
 ifneq (,$(wildcard finetune))
        $(CXX) $(CXXFLAGS) -c $< -o $(call GET_OBJ_FILE, $<)