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.

Sign up for GitHub

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

Jump to bottom

paket pack with enabled include-referenced-projects flag fails with NRE #1434

Closed
ar3cka opened this issue Jan 26, 2016 · 6 comments
Closed

paket pack with enabled include-referenced-projects flag fails with NRE #1434

ar3cka opened this issue Jan 26, 2016 · 6 comments
Labels
bug

Comments

@ar3cka
Copy link

ar3cka commented Jan 26, 2016

Find the build log here https://ci.appveyor.com/project/ar3cka/journalist
@forki forki added the bug label Jan 27, 2016
@forki
Copy link
Member

forki commented Jan 27, 2016

I can reproduce

forki added a commit that referenced this issue Jan 27, 2016
@forki
Fix NRE in #1434
d5b99ef
@forki
Copy link
Member

forki commented Jan 27, 2016

@pms1969 I fixed it in #1435, but I'm not completely sure why that was in there. Any ideas?

@forki
Copy link
Member

forki commented Jan 27, 2016

@ar3cka I released a new paket prerelease which allows you to test if the behaviour is correct. I mean the NRE should be gone, but you should look if the packages are built correctly.

forki added a commit that referenced this issue Jan 27, 2016
@forki
Fix NRE in #1434
3b3a73d
forki added a commit that referenced this issue Jan 27, 2016
@forki
Fix NRE in #1434
eaaa164
@pms1969
Copy link
Contributor

pms1969 commented Jan 27, 2016

ok, I'll check it out. and let you know.

@pms1969
Copy link
Contributor

pms1969 commented Jan 27, 2016

looks good.

forki added a commit that referenced this issue Jan 27, 2016
@forki
Merge pull request #1435 from fsprojects/fix-1434
0f2d99c 
Fix NRE in #1434
@forki forki closed this as completed in bdb22c8 Jan 27, 2016
@ar3cka
Copy link
Author

ar3cka commented Jan 28, 2016

I've checked. Package contains all assemblies. Thx.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@forki @ar3cka @pms1969