Skip to content

convert-to-nuget should not delete nuget.exe #320

Description

@haf

Unless it can also push to nuget, because that file might be in the source tree because I need to push the package.

Activity

  1. forki commented on Nov 1, 2014

    @forki
    Member

    I removed that manually for suave and added Nuget.commandline as a package
    in the dependencies file.

    So the path changes. I did that in the rake file.
    On Nov 1, 2014 6:06 PM, "Henrik Feldt" notifications@github.com wrote:

    Unless it can also push to nuget, because that file might be in the source
    tree because I need to push the package.

    —
    Reply to this email directly or view it on GitHub
    #320.

  2. haf commented on Nov 1, 2014

    @haf
    MemberAuthor

    Ah, I see, thank you. Perhaps consider this issue to be a 'warning' on deletion then?

    On 01 Nov 2014, at 18:19, Steffen Forkmann notifications@github.com wrote:

    I removed that manually for suave and added Nuget.commandline as a package
    in the dependencies file.

    So the path changes. I did that in the rake file.
    On Nov 1, 2014 6:06 PM, "Henrik Feldt" notifications@github.com wrote:

    Unless it can also push to nuget, because that file might be in the source
    tree because I need to push the package.

    —
    Reply to this email directly or view it on GitHub
    #320.

    —
    Reply to this email directly or view it on GitHub.

  3. forki commented on Nov 1, 2014

    @forki
    Member

    Yes we have to make this more obvious. I think tracing a warning is a good
    idea.
    On Nov 1, 2014 6:20 PM, "Henrik Feldt" notifications@github.com wrote:

    Ah, I see, thank you. Perhaps consider this issue to be a 'warning' on
    deletion then?

    On 01 Nov 2014, at 18:19, Steffen Forkmann notifications@github.com
    wrote:

    I removed that manually for suave and added Nuget.commandline as a
    package
    in the dependencies file.

    So the path changes. I did that in the rake file.
    On Nov 1, 2014 6:06 PM, "Henrik Feldt" notifications@github.com
    wrote:

    Unless it can also push to nuget, because that file might be in the
    source
    tree because I need to push the package.

    —
    Reply to this email directly or view it on GitHub
    #320.

    —
    Reply to this email directly or view it on GitHub.

    —
    Reply to this email directly or view it on GitHub
    #320 (comment).

  4. theimowski commented on Nov 1, 2014

    @theimowski
    Member

    IIRC, convert-from-nuget will add the Nuget.CommandLine package to dependencies if it removed Nuget.exe, but yes, a warning message to change the paths should be output

  5. forki commented on Nov 1, 2014

    @forki
    Member

    Yes, but suave had it in tools folder. So I did it manually.
    On Nov 1, 2014 6:29 PM, "Tomasz Heimowski" notifications@github.com wrote:

    IIRC, convert-from-nuget will add the Nuget.CommandLine package to
    dependencies if it removed Nuget.exe, but yes, a warning message to change
    the paths should be output

    —
    Reply to this email directly or view it on GitHub
    #320 (comment).

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions