fsprojects / Paket

A dependency manager for .NET with support for NuGet packages and Git repositories.
https://fsprojects.github.io/Paket/
MIT License
2.02k stars 520 forks source link

paket pack with CLI CPP projects #1870

Open pavel-khritonenko opened 8 years ago

pavel-khritonenko commented 8 years ago

Description

I can't create nuget package using paket.exe from CLI C++ library

Repro steps

I have ./.paket/ in my PATH environment variable

  1. Create empty C++ project (I used VS2015)
  2. run paket init (I actually run ../other-project/.paket/paket init)
  3. run paket.bootstrapper
  4. run paket install
  5. add paket.template with content type project
  6. run paket pack output .

    Expected behavior

Paket creates "empty" nuget package

Actual behavior

$ paket pack output . Paket version 3.15.0.0 Paket failed with: Unable to find Release output path node in file C:\projects\paket-cpp\ClassLibrary1\ClassLibrary1\ClassLibrary1.vcxproj for any known platforms

Related information

  • Latest Windows 10 (Red stone)
  • .NET 4.6.1

    Attached project (without binaries or paket.exe)

paket-cpp.zip

forki commented 8 years ago

how would you detect the output path in that file?

pavel-khritonenko commented 8 years ago

I changed outdir parameter in project options and now I have this change in .vcxproj:

image

By default it is $(SolutionDir)$(Platform)\$(Configuration)\

forki commented 8 years ago

So after the changes it pack works?

Am 23.08.2016 1:16 nachm. schrieb "pavelhritonenko" < notifications@github.com>:

I changed outdir in project options and now I have this change in .vcxproj:

[image: image] https://cloud.githubusercontent.com/assets/2510026/17890029/12aa484a-693c-11e6-8610-e397f96d1661.png

By default it is $(SolutionDir)$(Platform)\$(Configuration)\

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/fsprojects/Paket/issues/1870#issuecomment-241700279, or mute the thread https://github.com/notifications/unsubscribe-auth/AADgNCCCiLmSS40crjDEMfkJMw6jn4Krks5qitaRgaJpZM4Jqzpq .

pavel-khritonenko commented 8 years ago

No, it doesn't (even if I specified OutDir node for all possible platforms), error message is the same.

forki commented 8 years ago

I'm pushing a fix which would also check for OutDir (currently we look for OutputPath), but I guess we can't use the default. Not sure how we would do this

forki commented 8 years ago

can you please retry?

pavel-khritonenko commented 8 years ago

First, thanks for instant reaction, it's amazing.

Next error was related to absent <AssemblyName /> tag in the .vcxproj file, I've added it manually and now I get this error when I run paket pack output . buildplatform x64:

$ paket pack output . buildplatform x64
Paket version 3.15.1.0
Paket failed with:
        The input sequence was empty.
Parameter name: source
forki commented 8 years ago

can you please upload zip again?

forki commented 8 years ago

the newly changed one I mean

pavel-khritonenko commented 8 years ago

https://github.com/pavelhritonenko/paket-cpp - I noticed there is unnecessary level of folders however I decided to not touch it.

forki commented 8 years ago

and another release. please give it a try

pavel-khritonenko commented 8 years ago

Version from repository I linked:

$ paket pack output . buildplatform x64
Paket version 3.15.2.0
Paket failed with:
        Index was outside the bounds of the array.
forki commented 8 years ago

Since I can't build your project. Please commit output binaries as well

Am 23.08.2016 17:45 schrieb "pavelhritonenko" notifications@github.com:

Version from repository I linked:

$ paket pack output . buildplatform x64 Paket version 3.15.2.0 Paket failed with: Index was outside the bounds of the array.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/fsprojects/Paket/issues/1870#issuecomment-241777970, or mute the thread https://github.com/notifications/unsubscribe-auth/AADgNG_HNRNxKBVOu1A8z4ud-FbtpBK_ks5qixWIgaJpZM4Jqzpq .

forki commented 8 years ago

so that a clean checkout would produce that error

pavel-khritonenko commented 8 years ago

I've just commited binaries. Did you mean that?

Khritonenko@khritonenko MINGW64 /C/projects/paket-cpp (master)
$ git clean -fXd
Removing .paket/paket.exe

Khritonenko@khritonenko MINGW64 /C/projects/paket-cpp (master)
$ git clean -fxd

Khritonenko@khritonenko MINGW64 /C/projects/paket-cpp (master)
$ paket.bootstrapper.exe
Checking Paket version (downloading latest stable)...
Copying version 3.15.2 from cache.
Done.

Khritonenko@khritonenko MINGW64 /C/projects/paket-cpp (master)
$ paket pack output . buildplatform x64
Paket version 3.15.2.0
Paket failed with:
        Index was outside the bounds of the array.
forki commented 8 years ago

sorry for playing pingpong. with latest version (just released) I now see:

image

this seems to be correct, right?

forki commented 8 years ago

"Missing: Authors"