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 pack with CLI CPP projects #1870

Open
pavel-khritonenko opened this Issue Aug 23, 2016 · 17 comments

Comments

Projects
None yet
3 participants
@pavel-khritonenko

pavel-khritonenko commented Aug 23, 2016

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

This comment has been minimized.

Show comment
Hide comment
@forki

forki Aug 23, 2016

Member

how would you detect the output path in that file?

Member

forki commented Aug 23, 2016

how would you detect the output path in that file?

@pavel-khritonenko

This comment has been minimized.

Show comment
Hide comment
@pavel-khritonenko

pavel-khritonenko Aug 23, 2016

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

image

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

pavel-khritonenko commented Aug 23, 2016

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

image

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

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Aug 23, 2016

Member

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
#1870 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNCCCiLmSS40crjDEMfkJMw6jn4Krks5qitaRgaJpZM4Jqzpq
.

Member

forki commented Aug 23, 2016

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
#1870 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNCCCiLmSS40crjDEMfkJMw6jn4Krks5qitaRgaJpZM4Jqzpq
.

@pavel-khritonenko

This comment has been minimized.

Show comment
Hide comment
@pavel-khritonenko

pavel-khritonenko Aug 23, 2016

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

pavel-khritonenko commented Aug 23, 2016

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

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Aug 23, 2016

Member

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

Member

forki commented Aug 23, 2016

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

This comment has been minimized.

Show comment
Hide comment
@forki

forki Aug 23, 2016

Member

can you please retry?

Member

forki commented Aug 23, 2016

can you please retry?

@pavel-khritonenko

This comment has been minimized.

Show comment
Hide comment
@pavel-khritonenko

pavel-khritonenko Aug 23, 2016

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

pavel-khritonenko commented Aug 23, 2016

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

This comment has been minimized.

Show comment
Hide comment
@forki

forki Aug 23, 2016

Member

can you please upload zip again?

Member

forki commented Aug 23, 2016

can you please upload zip again?

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Aug 23, 2016

Member

the newly changed one I mean

Member

forki commented Aug 23, 2016

the newly changed one I mean

@pavel-khritonenko

This comment has been minimized.

Show comment
Hide comment
@pavel-khritonenko

pavel-khritonenko Aug 23, 2016

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

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

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Aug 23, 2016

Member

and another release. please give it a try

Member

forki commented Aug 23, 2016

and another release. please give it a try

@pavel-khritonenko

This comment has been minimized.

Show comment
Hide comment
@pavel-khritonenko

pavel-khritonenko Aug 23, 2016

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.

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

This comment has been minimized.

Show comment
Hide comment
@forki

forki Aug 23, 2016

Member

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
#1870 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNG_HNRNxKBVOu1A8z4ud-FbtpBK_ks5qixWIgaJpZM4Jqzpq
.

Member

forki commented Aug 23, 2016

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
#1870 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNG_HNRNxKBVOu1A8z4ud-FbtpBK_ks5qixWIgaJpZM4Jqzpq
.

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Aug 23, 2016

Member

so that a clean checkout would produce that error

Member

forki commented Aug 23, 2016

so that a clean checkout would produce that error

@pavel-khritonenko

This comment has been minimized.

Show comment
Hide comment
@pavel-khritonenko

pavel-khritonenko Aug 23, 2016

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.

pavel-khritonenko commented Aug 23, 2016

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

This comment has been minimized.

Show comment
Hide comment
@forki

forki Aug 23, 2016

Member

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

image

this seems to be correct, right?

Member

forki commented Aug 23, 2016

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

image

this seems to be correct, right?

@forki forki added the bug label Aug 23, 2016

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Aug 24, 2016

Member

"Missing: Authors"

Member

forki commented Aug 24, 2016

"Missing: Authors"

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