Move HTTP transport sources out of LLHTTP conditional

RahulHere · RahulHere · commit 2246e87ba2c7 · 2026-01-31T20:14:27.000+08:00
The http_sse_transport_socket.cc and https_sse_transport_factory.cc
files don't actually depend on llhttp - only llhttp_parser.cc does.
Moving them out of the conditional fixes Windows ARM64 build which
doesn't use llhttp.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -485,17 +485,17 @@ set(MCP_CLIENT_SERVER_SOURCES
     src/server/mcp_server.cc
 )
 
-# HTTP parser sources (conditional on llhttp)
+# HTTP parser sources (some conditional on llhttp)
 set(MCP_HTTP_SOURCES
     src/http/http_parser.cc
     src/http/sse_parser.cc
+    src/transport/http_sse_transport_socket.cc  # HTTP+SSE with layered architecture
+    src/transport/https_sse_transport_factory.cc  # HTTPS+SSE factory
 )
 
 if(LLHTTP_FOUND)
     list(APPEND MCP_HTTP_SOURCES
         src/http/llhttp_parser.cc
-        src/transport/http_sse_transport_socket.cc  # HTTP+SSE with layered architecture
-        src/transport/https_sse_transport_factory.cc  # HTTPS+SSE factory
     )
 endif()
 

Original file line number	Diff line number	Diff line change
`@@ -485,17 +485,17 @@ set(MCP_CLIENT_SERVER_SOURCES`
`485`	`485`	`src/server/mcp_server.cc`
`486`	`486`	`)`
`487`	`487`
`488`		`-# HTTP parser sources (conditional on llhttp)`
	`488`	`+# HTTP parser sources (some conditional on llhttp)`
`489`	`489`	`set(MCP_HTTP_SOURCES`
`490`	`490`	`src/http/http_parser.cc`
`491`	`491`	`src/http/sse_parser.cc`
	`492`	`+ src/transport/http_sse_transport_socket.cc # HTTP+SSE with layered architecture`
	`493`	`+ src/transport/https_sse_transport_factory.cc # HTTPS+SSE factory`
`492`	`494`	`)`
`493`	`495`
`494`	`496`	`if(LLHTTP_FOUND)`
`495`	`497`	`list(APPEND MCP_HTTP_SOURCES`
`496`	`498`	`src/http/llhttp_parser.cc`
`497`		`- src/transport/http_sse_transport_socket.cc # HTTP+SSE with layered architecture`
`498`		`- src/transport/https_sse_transport_factory.cc # HTTPS+SSE factory`
`499`	`499`	`)`
`500`	`500`	`endif()`
`501`	`501`