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

File already exists at path #89

Closed
Spreeuw opened this issue Nov 24, 2020 · 2 comments
Closed

File already exists at path #89

Spreeuw opened this issue Nov 24, 2020 · 2 comments

Comments

@Spreeuw
Copy link

Spreeuw commented Nov 24, 2020

Using latest master with this config:

{
    "require": {
        "iio/libmergepdf": "^4.0"
    },
    "require-dev": {
        "coenjacobs/mozart": "dev-master"
    },
    "extra": {
        "mozart": {
            "dep_namespace": "MyLibMerge\\Vendor",
            "dep_directory": "/lib/packages/",
            "classmap_directory": "/lib/classes/",
            "classmap_prefix": "MyLibMerge_",
            "excluded_packages": [
            ],
            "override_autoload": {
            },
            "delete_vendor_directories": true
        }
    },
    "scripts": {
        "post-install-cmd": [
            "\"vendor/bin/mozart\" compose",
            "composer dump-autoload"
        ],
        "post-update-cmd": [
            "\"vendor/bin/mozart\" compose",
            "composer dump-autoload"
        ]
    }
}

I get the error:

File already exists at path: lib/classes/tecnickcom/tcpdf/vendor/tecnickcom/tcpdf/tcpdf.php

This package tecnickcom/tcpdf is included as a dependency directly in the iio/libmergepdf package but also as a grandchild, via the setasign/fpdi dependency.

I can avoid this by adding it to the excluded_packages which is probably fine for this particular package, but wanted to report the bug for completeness.

There appears to be some sort of path duplication too: tecnickcom/tcpdf/vendor/tecnickcom (related to #58 but for classmaps?)

@BrianHenryIE
Copy link
Contributor

BrianHenryIE commented Nov 24, 2020

Hey, try this:

{
  "require": {
    "iio/libmergepdf": "^4.0"
  },
  "require-dev": {
    "coenjacobs/mozart": "dev-master",
    "cweagans/composer-patches": "*"
  },
  "extra": {
    "patches": {
      "coenjacobs/mozart": {
        "Move each file only once (classmap)": "https://github.com/coenjacobs/mozart/pull/91.patch"
      }
    },
    "mozart": {
      "dep_namespace": "MyLibMerge\\Vendor",
      "dep_directory": "/lib/packages/",
      "classmap_directory": "/lib/classes/",
      "classmap_prefix": "MyLibMerge_",
      "excluded_packages": [
      ],
      "override_autoload": {
      },
      "delete_vendor_directories": true
    }
  },
  "scripts": {
    "post-install-cmd": [
      "\"vendor/bin/mozart\" compose",
      "composer dump-autoload"
    ],
    "post-update-cmd": [
      "\"vendor/bin/mozart\" compose",
      "composer dump-autoload"
    ]
  }
}

@Spreeuw
Copy link
Author

Spreeuw commented Nov 25, 2020

Works for me! 💯

coenjacobs added a commit that referenced this issue Jan 17, 2021
BrianHenryIE added a commit to BrianHenryIE/strauss that referenced this issue May 1, 2021
szepeviktor pushed a commit to szepeviktor/BrianHenryIE_mozart that referenced this issue Apr 26, 2024
…vement

Add support for casting a namespaced class to a string.
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

No branches or pull requests

2 participants