Skip to content

Commit c506ea5

Browse files
committed
change naming
1 parent a2b330f commit c506ea5

2 files changed

Lines changed: 33 additions & 30 deletions

File tree

cmake/GrpcTargets.cmake

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -175,5 +175,9 @@ function(add_grpc_library NAME)
175175
add_library(${NAME} STATIC ${generated_sources} ${generated_usrv_sources})
176176
target_compile_options(${NAME} PUBLIC -Wno-unused-parameter)
177177
target_include_directories(${NAME} SYSTEM PUBLIC ${include_paths})
178-
target_link_libraries(${NAME} PUBLIC userver-grpc)
178+
if(USERVER_CONAN)
179+
target_link_libraries(${NAME} PUBLIC userver::grpc)
180+
else()
181+
target_link_libraries(${NAME} PUBLIC userver-grpc)
182+
endif()
179183
endfunction()

conanfile.py

Lines changed: 28 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -119,8 +119,6 @@ def requirements(self):
119119
self.requires('gtest/1.12.1')
120120
self.requires('benchmark/1.6.2')
121121

122-
123-
124122
def generate(self):
125123
tc = CMakeToolchain(self)
126124
tc.variables['CMAKE_FIND_DEBUG_MODE'] = False
@@ -281,58 +279,57 @@ def amqpcpp():
281279
return ["amqp-cpp::amqp-cpp"] if self.options.with_rabbitmq else []
282280

283281
userver_components = [
284-
{"target": "userver-core",
282+
{"target": "core",
285283
"lib": "core",
286-
"requires": ["userver-core-internal"] + fmt() + cctz() + boost() + concurrentqueue() + yaml() + libev() + http_parser() + curl() + cryptopp() + jemalloc() + ares() }
284+
"requires": ["core-internal"] + fmt() + cctz() + boost() + concurrentqueue() + yaml() + libev() + http_parser() + curl() + cryptopp() + jemalloc() + ares() }
287285
]
288286
if self.options.with_universal:
289287
userver_components.extend([
290-
{"target": "userver-universal",
288+
{"target": "universal",
291289
"lib": "universal",
292290
"requires": fmt() + cctz() + boost() + concurrentqueue() + yaml() + cryptopp() + jemalloc() + openssl() }
293291
])
294292
if self.options.with_grpc:
295293
userver_components.extend([
296-
{"target": "userver-grpc",
294+
{"target": "grpc",
297295
"lib": "grpc",
298-
"requires": ["userver-core"] + grpc() },
299-
{"target":
300-
"userver-api-common-protos",
296+
"requires": ["core"] + grpc() },
297+
{"target": "api-common-protos",
301298
"lib": "api-common-protos",
302-
"requires": ["userver-grpc"] }
299+
"requires": ["grpc"] }
303300
])
304301
if self.options.with_utest:
305302
userver_components.extend([
306-
{"target": "userver-utest",
303+
{"target": "utest",
307304
"lib": "utest",
308-
"requires": ["userver-core"] + gtest() },
309-
{"target": "userver-ubench",
305+
"requires": ["core"] + gtest() },
306+
{"target": "ubench",
310307
"lib": "ubench",
311-
"requires": ["userver-core"] + benchmark() }
308+
"requires": ["core"] + benchmark() }
312309
])
313310
if self.options.with_postgresql:
314311
userver_components.extend([
315-
{"target": "userver-postgresql",
312+
{"target": "postgresql",
316313
"lib": "postgresql",
317-
"requires": ["userver-core"] + postgresql() }
314+
"requires": ["core"] + postgresql() }
318315
])
319316
if self.options.with_mongodb:
320317
userver_components.extend([
321-
{"target": "userver-mongo",
318+
{"target": "mongo",
322319
"lib": "mongo",
323-
"requires": ["userver-core"] + mongo() }
320+
"requires": ["core"] + mongo() }
324321
])
325322
if self.options.with_redis:
326323
userver_components.extend([
327-
{"target": "userver-redis",
324+
{"target": "redis",
328325
"lib": "redis",
329-
"requires": ["userver-core"] + hiredis() }
326+
"requires": ["core"] + hiredis() }
330327
])
331328
if self.options.with_rabbitmq:
332329
userver_components.extend([
333-
{"target": "userver-rabbitmq",
330+
{"target": "rabbitmq",
334331
"lib": "rabbitmq",
335-
"requires": ["userver-core"] + amqpcpp() }
332+
"requires": ["core"] + amqpcpp() }
336333
])
337334
return userver_components
338335

@@ -353,7 +350,7 @@ def add_components(components):
353350
requires = component["requires"]
354351
# TODO: we should also define COMPONENTS names of each target for find_package() but not possible yet in CMakeDeps
355352
# see https://github.com/conan-io/conan/issues/10258
356-
self.cpp_info.components[conan_component].set_property("cmake_target_name", cmake_target)
353+
self.cpp_info.components[conan_component].set_property("cmake_target_name", "userver::" + cmake_target)
357354
self.cpp_info.components[conan_component].libs = [lib_name]
358355
if cmake_component=="core":
359356
self.cpp_info.components[conan_component].libs.append(get_lib_name("core-internal"))
@@ -362,26 +359,28 @@ def add_components(components):
362359
if cmake_component=="core" or cmake_component=="universal":
363360
self.cpp_info.components[conan_component].includedirs.append(os.path.join("include", "shared"))
364361

362+
print(component)
363+
print(requires)
365364
self.cpp_info.components[conan_component].requires = requires
366365

367366

368-
self.cpp_info.components["userver-core-internal"].defines.append(
367+
self.cpp_info.components["core-internal"].defines.append(
369368
f'USERVER_NAMESPACE={self.options.namespace}',
370369
)
371-
self.cpp_info.components["userver-core-internal"].defines.append(
370+
self.cpp_info.components["core-internal"].defines.append(
372371
f'USERVER_NAMESPACE_BEGIN={self.options.namespace_begin}',
373372
)
374-
self.cpp_info.components["userver-core-internal"].defines.append(
373+
self.cpp_info.components["core-internal"].defines.append(
375374
f'USERVER_NAMESPACE_END={self.options.namespace_end}',
376375
)
377376

378-
self.cpp_info.components["userver-universal"].defines.append(
377+
self.cpp_info.components["universal"].defines.append(
379378
f'USERVER_NAMESPACE={self.options.namespace}',
380379
)
381-
self.cpp_info.components["userver-universal"].defines.append(
380+
self.cpp_info.components["universal"].defines.append(
382381
f'USERVER_NAMESPACE_BEGIN={self.options.namespace_begin}',
383382
)
384-
self.cpp_info.components["userver-universal"].defines.append(
383+
self.cpp_info.components["universal"].defines.append(
385384
f'USERVER_NAMESPACE_END={self.options.namespace_end}',
386385
)
387386

0 commit comments

Comments
 (0)