Skip to content

Renamed all the base_source_folder -> export_sources_folder - #2514

Merged
czoido merged 2 commits into
conan-io:developfrom
franramirez688:feature/export_sources_folder
Apr 28, 2022
Merged

czoido merged 2 commits into
conan-io:developfrom
franramirez688:feature/export_sources_folder

Conversation

@franramirez688

@franramirez688 franramirez688 commented Apr 28, 2022

Copy link
Copy Markdown
Contributor

Docs for conan-io/conan#11055

Coming from the original Conan PR conan-io/conan#11055:

Changelog: Fix: Renamed self.base_source_folder to self.export_source_folder. That variable was introduced to reference the folder where the export_sources are. Currently, they are copied to the source folder but might be changed in the future to avoid copying them, so self.export_source_folder will always point to the exports_sources.

@SSE4

SSE4 commented Apr 28, 2022

Copy link
Copy Markdown
Contributor

it lacks some motivation behind the rename, could you add some notes on why was it needed, or what kind of problem were you trying to solve by this rename?

@czoido
czoido merged commit 3ee1fac into conan-io:develop Apr 28, 2022
czoido added a commit that referenced this pull request May 3, 2022
* Remove reference to master banch (#2497)

It no longer exists in conan-io/consan repository

* Added conf information about new apple flags (#2506)

* Fix typo error for os_version in cmake.rst (#2515)

Fix: #2513

* Renamed all the base_source_folder -> export_sources_folder (#2514)

* Renamed all the base_source_folder -> export_sources_folder

* Update package_layout.rst

* Docs for 11112 and 11117 and 11103 (#2512)

* Docs for 11112 and 11117

* Update reference/conanfile/tools/cmake/cmaketoolchain.rst

Co-authored-by: Francisco Ramírez <franchuti688@gmail.com>

* Simplified Build Preset name

Co-authored-by: Francisco Ramírez <franchuti688@gmail.com>

* [CMakeDeps] Added cmake_set_interface_link_directories property info (#2510)

* Added cmake_set_interface_link_directories property info

* Update cmakedeps.rst

* Update reference/conanfile/tools/cmake/cmakedeps.rst

Co-authored-by: Luis Martinez <lasote@gmail.com>

* Update cmakedeps.rst

* Update cmakedeps.rst

Co-authored-by: Luis Martinez <lasote@gmail.com>

* Docs for new autotools templates (#2511)

* add templates

* change argument

* [MSBuildToolchain][conf] Adding extra flags configurations to `MSBuildToolchain` (#2507)

* Added conf information

* Update microsoft.rst

* Update reference/conanfile/tools/microsoft.rst

Co-authored-by: Carlos Zoido <mrgalleta@gmail.com>

* Added env info to self.run() command (#2516)

* powershell env docs (#2517)

* powershell env docs

* added warning about env and PATH limits

* fixed CI

* some notes about relocatable libs (#2518)

* Docs: Add PKG_CONFIG_PATH to AutotoolsToolchain (#2509)

* add docs for PKG_CONFIG_PATH in toolchain

* Update reference/conanfile/tools/gnu/autotoolstoolchain.rst

Co-authored-by: Francisco Ramírez <franchuti688@gmail.com>

Co-authored-by: Luis Martinez <lasote@gmail.com>
Co-authored-by: Francisco Ramírez <franchuti688@gmail.com>

* conan 1.48.0

* Example for conanrun (#2519)

* Added example for conanrun launcher

* typo

* Added comments

* Added complete example

* Minimized example

* typo

* Rephrased

* removed path as value

Co-authored-by: Jeremy Lundy <16428663+jlundy2@users.noreply.github.com>
Co-authored-by: Francisco Ramírez <franchuti688@gmail.com>
Co-authored-by: Luca Bonifacio <4151271+lucabonifacio@users.noreply.github.com>
Co-authored-by: Luis Martinez <lasote@gmail.com>
Co-authored-by: James <james@conan.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants