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

Dub with inexistent dependency on other configuration #2627

Closed
MrcSnm opened this issue Apr 1, 2023 · 2 comments
Closed

Dub with inexistent dependency on other configuration #2627

MrcSnm opened this issue Apr 1, 2023 · 2 comments

Comments

@MrcSnm
Copy link
Contributor

MrcSnm commented Apr 1, 2023

System information

  • dub version: 1.31.0
  • OS Platform and distribution: Any
  • compiler version 2.102

Bug Description

Basically, this will fail when build with the default configuration. But the non existent dependency in the other config is not required for the default configuration and thus, should build.
I know that there exists optional, but optional seems to have a little different use case than handling that. I want to know if you guys are willing to keep it like that and I must really use optional or if that is a bug.

How to reproduce?

{
	"authors": [
		"Marcelo Silva"
	],
	"copyright": "Copyright © 2023, Marcelo Silva",
	"description": "A minimal D application.",
    "configurations": [
        {
            "name": "default"
        },
        {
            "name": "withDeps",
            "dependencies": {
                "noExist": {"path": "../"}
            }
        }
    ],
	"license": "proprietary",
	"name": "inexistent_ddep"
}

Expected Behavior

Logs

@WebFreak001
Copy link
Member

see #1217 for that dependency resolution should fail right now. Building should theoretically be fine though, so we can maybe adjust the building / dependency fetching algorithm to only fetch dependencies that are needed for the build configuration.

@Geod24
Copy link
Member

Geod24 commented Jan 17, 2024

As mentioned, it's a duplicate of a well known issue (which is even pinned in this repository). I am working on it.

@Geod24 Geod24 closed this as completed Jan 17, 2024
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