Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[feature] use short_paths during build, normal paths during packaging #6414

Closed
1 task done
madebr opened this issue Jan 24, 2020 · 4 comments
Closed
1 task done

[feature] use short_paths during build, normal paths during packaging #6414

madebr opened this issue Jan 24, 2020 · 4 comments

Comments

@madebr
Copy link
Contributor

madebr commented Jan 24, 2020

Some packages need short_paths enabled during build, but once built can be stored normally in a package folder in CONAN_HOME.
For example conan-io/conan-center-index#725,
this package needs short_paths enabled during build() but can have a normal path during package().

Essentially, it would be nice if self.build_folder can be a short path but self.package_folder a normal path.

@madebr madebr changed the title [feature] se short_paths during build, use normal paths for packaging [feature] use short_paths during build, normal paths during packaging Jan 24, 2020
@memsharded
Copy link
Member

The short_paths feature is a bit of a mess, complicated and fragile. I am proposing in https://github.com/conan-io/conan/pull/6415/files some possibility to try to improve it.

This feature request makes sense, but I am a bit afraid of adding more complexity and fragility into the code, for something that doesn't seem critical, but a nice to have. I would delay this to see if https://github.com/conan-io/conan/pull/6415/files moves forward, and come back later to analyze if makes sense or not. A possibility could be allowing defining short_paths = "build", and short_paths = "build", "package" or something like that.

@uilianries
Copy link
Member

A possibility could be allowing defining short_paths = "build", and short_paths = "build", "package"

It sounds good, any user will be able to decide what they want.

@madebr
Copy link
Contributor Author

madebr commented Jan 25, 2020

A possibility could be allowing defining short_paths = "build", and short_paths = "build", "package" or something like that.

This would be nice.
Also, I think adding source as an option would be interesting. This might benefit no_copy_source = True builds.

@memsharded
Copy link
Member

short_paths removed in Conan 2.0, closing this ticket as outdated.

@memsharded memsharded closed this as not planned Won't fix, can't repro, duplicate, stale Sep 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants