Skip to content

Add documentation for the ProduceOnlyReferenceAssembly msbuild option#29404

Merged
chsienki merged 1 commit intodotnet:masterfrom
chsienki:ref_only_docs
Aug 20, 2018
Merged

Add documentation for the ProduceOnlyReferenceAssembly msbuild option#29404
chsienki merged 1 commit intodotnet:masterfrom
chsienki:ref_only_docs

Conversation

@chsienki
Copy link
Copy Markdown
Member

Documents the ProduceOnlyReferenceAssembly option in the refout docs

@chsienki
Copy link
Copy Markdown
Member Author

I've also opened an issue for on docs (https://github.com/MicrosoftDocs/visualstudio-docs/issues/1407) to get it added to the common msbuild project properties section when it becomes available.

@chsienki
Copy link
Copy Markdown
Member Author

@dotnet/roslyn-compiler for review.

Comment thread docs/features/refout.md

* `ProduceReferenceAssembly` (boolean) controls whether to create the item passed to the compiler task (and thus pass `/refout:`). It requires opt-in. It is recommended that `Deterministic` also be set for best result (see details above).
* `ProduceReferenceAssembly` (boolean) controls whether to create the item passed to the compiler task (and thus pass `/refout:`). It requires opt-in. It is recommended that `Deterministic` also be set for best result (see details above). Cannot be used in conjunction with `ProduceOnlyReferenceAssembly`
* `ProduceOnlyReferenceAssembly` (boolean) controls whether to pass `/refonly` to the compiler. It requires opt-in. Cannot be used in conjunction with `ProduceReferenceAssembly`
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit: inconsistency on whether or not you put a colon after the option name.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/refout: takes a path as an option. /refonly is just a switch with nothing passed to it.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah yes

@jcouv jcouv added this to the 16.0 milestone Aug 20, 2018
@chsienki chsienki merged commit 233957b into dotnet:master Aug 20, 2018
333fred added a commit to 333fred/roslyn that referenced this pull request Aug 23, 2018
…atures/compiler

* dotnet/features/compiler: (183 commits)
  Document dotnet#13686 (dotnet#29416)
  Fix xml doc-comment resolution warnings
  PR feedback
  Add a perf benchmark for the compiler Emit phase (dotnet#29323)
  PR feedback
  Add documentation for the ProduceOnlyReferenceAssembly msbuild option to the refout docs (dotnet#29404)
  PR feedback
  Address PR feedback
  Expose refonly as an msbuild property (dotnet#29352)
  Add refactoring to convert foreach loop into linq invocations (Select, SelectMany and Where invocations)
  disable bad test
  Fix build error
  Restore the machinery for enabling language features by feature flags
  Assert message should sort diagnostics when possible (dotnet#29151)
  Update unix contributing docs
  Added Newtonsoft.Json to exclusion list
  Fix generation of compiler binding redirects (dotnet#29331)
  fixing hintspan size when Sub has attributes (dotnet#29115)
  Use Microsoft.SymbolUploader build task for symbol publishing (dotnet#29247)
  Remove default handling for bound nodes in (CS/VB)OperstionFactory.CreateInternal. (dotnet#29258)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants