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

Problem with source code content #1592

Closed
tsibelman opened this Issue Apr 11, 2016 · 6 comments

Comments

Projects
None yet
2 participants
@tsibelman
Contributor

tsibelman commented Apr 11, 2016

Description

If nugget has source code as content item, paket should not add CopyToOutputDirectory Always element

Example:

<Compile Include="Properties\orleans.codegen.cs">
      <CopyToOutputDirectory>Always</CopyToOutputDirectory>
 </Compile>

@tsibelman

This comment has been minimized.

Show comment
Hide comment
@tsibelman

tsibelman Apr 13, 2016

Contributor

@forki ?

Contributor

tsibelman commented Apr 13, 2016

@forki ?

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Apr 13, 2016

Member

Do you have a repro sample for us? Or a package that creates such issues?
On Apr 13, 2016 08:18, "tsibelman" notifications@github.com wrote:

@forki https://github.com/forki ?


You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#1592 (comment)

Member

forki commented Apr 13, 2016

Do you have a repro sample for us? Or a package that creates such issues?
On Apr 13, 2016 08:18, "tsibelman" notifications@github.com wrote:

@forki https://github.com/forki ?


You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#1592 (comment)

@tsibelman

This comment has been minimized.

Show comment
Hide comment
@tsibelman

tsibelman Apr 13, 2016

Contributor

Yes you can try with Microsoft.Orleans.Templates.Interfaces nuget

Contributor

tsibelman commented Apr 13, 2016

Yes you can try with Microsoft.Orleans.Templates.Interfaces nuget

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Apr 13, 2016

Member

but you explicitly specified to set that property right?

Member

forki commented Apr 13, 2016

but you explicitly specified to set that property right?

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Apr 13, 2016

Member

I guess we should never set that for Compile items

Member

forki commented Apr 13, 2016

I guess we should never set that for Compile items

@tsibelman

This comment has been minimized.

Show comment
Hide comment
@tsibelman

tsibelman Apr 13, 2016

Contributor

I used content once for this nugget, I think you correct that Compile items never should be copied

Contributor

tsibelman commented Apr 13, 2016

I used content once for this nugget, I think you correct that Compile items never should be copied

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