Pre-calc Paths

Author: Behemyth

cppython/plugins/cmake/plugin.py

@@ -24,6 +24,9 @@ def __init__(self, group_data: GeneratorPluginGroupData, core_data: CorePluginDa
         self.data = resolve_cmake_data(data, core_data)
         self.builder = Builder()
 
+        self._cppython_preset_directory = self.core_data.cppython_data.tool_path / 'cppython'
+        self._provider_directory = self._cppython_preset_directory / 'providers'
+
     @staticmethod
     def features(_: Path) -> SupportedGeneratorFeatures:
         """Queries if CMake is supported
@@ -58,16 +61,13 @@ def sync(self, sync_data: SyncData) -> None:
             sync_data: The input data
         """
         if isinstance(sync_data, CMakeSyncData):
-            cppython_preset_directory = self.core_data.cppython_data.tool_path / 'cppython'
-            cppython_preset_directory.mkdir(parents=True, exist_ok=True)
-
-            provider_directory = cppython_preset_directory / 'providers'
-            provider_directory.mkdir(parents=True, exist_ok=True)
+            self._cppython_preset_directory.mkdir(parents=True, exist_ok=True)
+            self._provider_directory.mkdir(parents=True, exist_ok=True)
 
-            self.builder.write_provider_preset(provider_directory, sync_data)
+            self.builder.write_provider_preset(self._provider_directory, sync_data)
 
             cppython_preset_file = self.builder.write_cppython_preset(
-                cppython_preset_directory, provider_directory, sync_data
+                self._cppython_preset_directory, self._provider_directory, sync_data
             )
 
             self.builder.write_root_presets(self.data.preset_file, cppython_preset_file)

cppython/plugins/conan/plugin.py

@@ -27,6 +27,11 @@ def __init__(
         self.core_data: CorePluginData = core_data
         self.data: ConanData = resolve_conan_data(configuration_data, core_data)
 
+        self._provider_file = self.core_data.cppython_data.tool_path / 'conan' / 'conan_provider.cmake'
+        self._provider_url = (
+            'https://raw.githubusercontent.com/conan-io/cmake-conan/refs/heads/develop2/conan_provider.cmake'
+        )
+
     @staticmethod
     def _download_file(url: str, file: Path) -> None:
         """Replaces the given file with the contents of the url"""
@@ -59,20 +64,16 @@ def information() -> Information:
 
     def install(self) -> None:
         """Installs the provider"""
-        conan_provider = self.core_data.cppython_data.tool_path / 'conan' / 'conan_provider.cmake'
-
         self._download_file(
-            'https://raw.githubusercontent.com/conan-io/cmake-conan/refs/heads/develop2/conan_provider.cmake',
-            conan_provider,
+            self._provider_url,
+            self._provider_file,
         )
 
     def update(self) -> None:
         """Updates the provider"""
-        conan_provider = self.core_data.cppython_data.tool_path / 'conan' / 'conan_provider.cmake'
-
         self._download_file(
-            'https://raw.githubusercontent.com/conan-io/cmake-conan/refs/heads/develop2/conan_provider.cmake',
-            conan_provider,
+            self._provider_url,
+            self._provider_file,
         )
 
     @staticmethod
@@ -87,8 +88,7 @@ def supported_sync_type(sync_type: type[SyncData]) -> bool:
         """
         return sync_type in CMakeGenerator.sync_types()
 
-    @staticmethod
-    def sync_data(consumer: SyncConsumer) -> SyncData:
+    def sync_data(self, consumer: SyncConsumer) -> SyncData:
         """_summary_
 
         Args:
@@ -99,6 +99,6 @@ def sync_data(consumer: SyncConsumer) -> SyncData:
         """
         for sync_type in consumer.sync_types():
             if sync_type == CMakeSyncData:
-                return CMakeSyncData(provider_name=TypeName('conan'), top_level_includes=Path('test'))
+                return CMakeSyncData(provider_name=TypeName('conan'), top_level_includes=self._provider_file)
 
         raise NotSupportedError('OOF')