Skip to content

Change for out-of-source build#5213

Closed
avtikhon wants to merge 1 commit intocurl:masterfrom
avtikhon:avtikhon/fix-for-out-of-source-build
Closed

Change for out-of-source build#5213
avtikhon wants to merge 1 commit intocurl:masterfrom
avtikhon:avtikhon/fix-for-out-of-source-build

Conversation

@avtikhon
Copy link
Copy Markdown
Contributor

We use to build Curl sources in our project and wanted to have out-of-source build. To make it able we need to avoid of use the current paths. In Curl sources we found this single place where for cleanup the current path was used for temporary files. We can suggest to avoid of its use and run 'find' command to find and remove needed files.
Need for tarantool/tarantool#4874

@bagder
Copy link
Copy Markdown
Member

bagder commented Apr 10, 2020

I don't understand the description. You want to fix out-of-tree builds. But why is storing a temp file in "current directory" a problem for those? In most cases, that's exactly where you want the files to be made as that's the place you build as compared to where the source lives.

I'm not against the fix, I like avoiding a tempfile, I just have a problem to understand the explanation!

@bagder bagder added the build label Apr 10, 2020
@bagder bagder closed this in 1e41bec Apr 15, 2020
@bagder
Copy link
Copy Markdown
Member

bagder commented Apr 15, 2020

Thanks. I modified the commit message before merging because of what I mentioned above.

avtikhon added a commit to tarantool/curl that referenced this pull request May 23, 2020
Copied from: "Fixed in 7.70.0 - April 29 2020"
  curl#5213

Closes curl#5213
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

2 participants