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

Paket version 5.157.0-alpha003 is breaking Fulma repo #3180

Closed
MangelMaxime opened this issue May 2, 2018 · 16 comments
Closed

Paket version 5.157.0-alpha003 is breaking Fulma repo #3180

MangelMaxime opened this issue May 2, 2018 · 16 comments
Assignees

Comments

@MangelMaxime
Copy link

Description

Since yesterday evening people are reporting me problems with Fulma repo and templates.

Errors are like:

> No executable found matching command "dotnet-fake"

> No executable found matching command "dotnet-fable"

I suspect this issues will impact any project using clitool.

Repro steps

Please provide the steps required to reproduce the problem

  1. git clone [email protected]:MangelMaxime/Fulma.git

  2. ./fake.sh build

  3. The build is failing

Expected behavior

dotnet fake should work.

Actual behavior

> No executable found matching command "dotnet-fake"

Known workarounds

If replace the magic mode with paket.exe version 5.156.7 then it's working again.

@forki
Copy link
Member

forki commented May 2, 2018

urgs @matthid

@matthid
Copy link
Member

matthid commented May 2, 2018

Oo? Is the paket bootstrapper using prereleases by default?

@MangelMaxime
Copy link
Author

@matthid I don't think.

I have this paket.exe.config file in my project:

<?xml version="1.0" encoding="utf-8" ?>
<configuration>
  <appSettings>
    <add key="Prerelease" value="True"/>
  </appSettings>
</configuration>

I should probably remove it at least from the template indeed but it's useful to detect bugs :)

@matthid
Copy link
Member

matthid commented May 2, 2018

Yes I’m glad we found this, but we shouldn’t break people on an alpha release :)

Is this an urgent fix considering the template it out? If yes @forki can you release another alpha from master? I’m not sure I can takle this today or tomorrow...

@forki
Copy link
Member

forki commented May 2, 2018

do we know what broke it? and how to fix?

@MangelMaxime
Copy link
Author

Just for info, I fixed the template and my Fulma repo to use only stable version of paket.

@matthid
Copy link
Member

matthid commented May 2, 2018

At least I have no idea but fest step would be to look atthe build output which I cannot run at the moment

@forki
Copy link
Member

forki commented May 2, 2018

so what should I do? people are reporting this on fable gitter? unlisting the prerelease versions?

@MangelMaxime
Copy link
Author

@forki I think the safe approach is to remove the prerelease versions.

@forki
Copy link
Member

forki commented May 2, 2018

@matthid would that break FAKE?

@MangelMaxime
Copy link
Author

Or apply the solution proposed by @matthid

If yes @forki can you release another alpha from master? I’m not sure I can takle this today or tomorrow...

@matthid
Copy link
Member

matthid commented May 2, 2018

No it won’t break fake. actually unlisting should have worked just as well, sorry

@forki
Copy link
Member

forki commented May 2, 2018

pushed a master version as latest prerelease

@matthid
Copy link
Member

matthid commented May 13, 2018

Ok I debugged into this, however now I have no idea why it ever worked. Apparently we currently don't detect these kind of new-style sdk projects (using Import instead of Sdk attribute as well as .proj file extension)
I fixed these issues and the problems went away. However it must be some kind of side-effect from some other change as all my changes were unrelated to this....

@matthid
Copy link
Member

matthid commented May 13, 2018

@MangelMaxime Can you try again with https://www.nuget.org/packages/Paket.Core/5.162.0-alpha002? Seems to work now.

@forki forki closed this as completed in be7f475 May 14, 2018
@MangelMaxime
Copy link
Author

Sorry for the delay.
This is working with latest version of paket.

Thank you guys

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

3 participants