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

[bug] '/' path separator is not handled correctly on Windows by Conan 1.30.0 #7856

Closed
db4 opened this issue Oct 9, 2020 · 3 comments · Fixed by #7861
Closed

[bug] '/' path separator is not handled correctly on Windows by Conan 1.30.0 #7856

db4 opened this issue Oct 9, 2020 · 3 comments · Fixed by #7861

Comments

@db4
Copy link
Contributor

db4 commented Oct 9, 2020

Environment Details (include every applicable attribute)

  • Operating System+version: Windows 10
  • Conan version: 1.30.0
  • Python version: 3.8.6

Steps to reproduce

The following consumer recipe illustrates the problem:

from conans import ConanFile
import os

class TestConan(ConanFile):
    requires = "zlib/1.2.11@conan/stable"

    def imports(self):
        pkg = "zlib"
        path = "licenses/LICENSE"
        self.copy(path, dst=pkg, root_package=pkg)
        if not os.path.exists(os.path.join(pkg, path)):
            raise Exception("Import failure!")

It works with 1.29.2 but fails with 1.30.0. If I replace / with \\ it works with 1.30.0 as well.

@DoDoENT
Copy link
Contributor

DoDoENT commented Oct 9, 2020

I was just about to report that. In my case, the following does not work in my conan package:

    def imports(self):
        self.copy('some-folder/*', root_package='some-other-package')

The some-folder gets imported on mac and Linux, but not on Windows. Downgrade to conan v1.29.2 solved the issue.

@czoido czoido self-assigned this Oct 9, 2020
@czoido
Copy link
Contributor

czoido commented Oct 9, 2020

Hi @db4, @DoDoENT,
Thanks a lot for reporting. I have been able to reproduce the issue, it was probably introduced by #7704
We'll fix as soon as possible.

@memsharded
Copy link
Member

Fixed by #7861, will be released in 1.30.2

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