Skip to content

v2.3.0 always selects vibe-d:core instead of vibe-core #201

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

Closed
schveiguy opened this issue Sep 28, 2019 · 6 comments
Closed

v2.3.0 always selects vibe-d:core instead of vibe-core #201

schveiguy opened this issue Sep 28, 2019 · 6 comments

Comments

@schveiguy
Copy link
Collaborator

I had to comment out this line: ec85195

Can we leave that commented out now, or is it absolutely needed?

@itkfm
Copy link

itkfm commented Sep 30, 2019

Deprecation: the vibe-d:core package is deprecated and will be removed in January 2020. Please update your code to depend on vibe-core instead.

@schveiguy
Copy link
Collaborator Author

Yeah, that's the message that clued me in that it's not selecting the right core. But I didn't think to look at the date! Certainly, this needs to be fixed soon. Maybe a version 3 of mysql-native requires vibe-core and doesn't support vibe-d:core? @Abscissa any thoughts?

@Abscissa
Copy link

Abscissa commented Oct 1, 2019

Been out-of-the-loop awhile as my day job got absurdly busy over the summer, so I haven't had a chance to refresh my memory on this stuff yet (plus I just got a new computer I'm still setting up) Things have mostly calmed back down now, so I should be able to dive back in again soon...

What I can say right now is this: My only concerns with this change are:

  1. making sure not to break OSX in the process (I think this is the issue that caused the offending line to be added in the first place: Travis OSX: Outgoing TCP conn refused with 0.8.3 (but ok on 0.7.32) vibe-d/vibe.d#2155 ). And:

  2. not breaking TOO much backwards compatibility. (I'm fine with a v3 that deprecates some older versions of dmd/vibe if necessary, depending how much).

What we need first is a simple PR that removes the offending line so travis-ci can tell us exactly what the breakages are (if any). Then we can go from there and either fix/deprecate those broken configurations as appropriate.

@schveiguy
Copy link
Collaborator Author

I'll make a simple removal PR and see what happens. I actually wanted to add a mechanism to clean up unused mysql connections, but I ran into this first (was developing using 2.2.1 which didn't have this issue).

@Kozzi11
Copy link

Kozzi11 commented Oct 8, 2019

I would say this is duplicate of #199

@Abscissa
Copy link

Now fixed. Mysqln master is now fully switched from "vibe-d:core" to "vibe-core".

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

4 participants