Skip to content

Conversation

pmiossec
Copy link
Member

Pro: The size of the package (slightly lighter, env 2Mo less)
Cons:

  • When you update the nuget packages, the references are added again
  • Seems to work but could not guarantee that there is not a missing assembly for some specific feature and that it won't crash

This PR is to open the discussion...

@spraints
Copy link
Member

If it compiles without them, I think it'll be fine.

When you update the nuget packages, the references are added again

What do you think would resolve this? Maybe a document about "how to update the TFS dependencies"? Or maybe a script to scrub the unnecessary references from the project file?

@pmiossec
Copy link
Member Author

What do you think would resolve this? Maybe a document about "how to update the TFS dependencies"? Or maybe a script to scrub the unnecessary references from the project file?

For me, the best solution is perhaps to use paket. A good introduction: http://forki.github.io/PaketIntro

@pmiossec pmiossec force-pushed the tfs2015_light_package branch from 82617a7 to ed39a8f Compare February 17, 2016 00:06
@spraints
Copy link
Member

paket sounds awesome. 👍 on switching to it.

👍 on this branch. It'd be nice to have some docs, but that doesn't need to hold this PR up.

pmiossec added a commit that referenced this pull request Feb 18, 2016
Remove useless assemblies to reduce release package size
@pmiossec pmiossec merged commit 929872b into git-tfs:master Feb 18, 2016
@pmiossec pmiossec deleted the tfs2015_light_package branch February 18, 2016 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants