Added BootstrapperOutputDir to config #3733
Conversation
|
Nvm, figured it out. Not used to unix filesystem.. :) |
docs/content/bootstrapper.md
Outdated
| * `Prerelease`: Same as `prerelease` option. | ||
| * `PaketVersion`: Same as `<version>` option. | ||
| * `BootstrapperOuputDir`: Same as `--output-dir=` option. |
docs/content/bootstrapper.md
Outdated
|
|
||
| **IMPORTANT PERFOMANCE NOTE:** | ||
|
|
||
| If you have aggresive Anti Virus monitoring, but have your development folder |
There was a problem hiding this comment.
Where it scans activity of processes in a non-excluded directory. Performance really suffers then if loads of files are created (which paket does a lot :) )
There was a problem hiding this comment.
But yeah, I agree, it could be worded differently, or even left out.
There was a problem hiding this comment.
I hope the updated wording is better - as well as having fewer typos... ;)
|
I think the advice should be that you configure a place that you can exclude from the antivirus check |
|
Sure, but why not exclude something in the temporary directory? |
|
I dunno. but I assume a gazillion of programs want to write there. So using a folder that only paket's config knows might be safer!? |
|
Just make a directory within the temp directory, this one can be excluded and is paket only? |
|
Well the issue then becomes a matter of rights, and targeted attempts. This will then be known vulnerability, and having write access to temp, can then circumvent AV by just writing to that folder. When it is configureable, it is not as simple to use it for direct attacks. :) - but yeah, would be a lot simpler to just exclude that folder, but not always as easy in enterprisy setups.. ;) |
|
I think with the docs and typo fixes we can take it. |
|
new bootstrapper is released. thanks |
Attempt at fixing the AV related performance issue mentioned in #3661 .
This is done by allowing
OutputDirto be set in AppSettings as well, so it can be included as a permanent setting for a repository.Since magic mode recommends using paket.exe.config for AppSettings, I've renamed the setting to
BootstrapperOutputDirin AppSettings, so it is clear that the output dir relates to the Bootstrapper only, and keeps some naming overlap from the same option provided to command line already.I've also added testing and minor doc updates.
Fixes #3661