package-validator icon indicating copy to clipboard operation
package-validator copied to clipboard

Guideline: Check for copying to program files and flag

Open ferventcoder opened this issue 10 years ago • 3 comments

Nothing should have a need to copy to program files without a really good reason. The native installer should make the changes required.

However if someone is just deciding that is where their portable application goes, that is an anti-pattern.

ferventcoder avatar Nov 25 '15 18:11 ferventcoder

@ferventcoder what was going to be your approach here? There are a number of ways to copy files, and simply looking for usage of Program Files/Program Files (x86) might not be the best way.

Thoughts?

gep13 avatar Dec 06 '15 20:12 gep13

I didn't have a best way yet, it was more linked to looking for use of both program files and any use of copying.

ferventcoder avatar Dec 07 '15 13:12 ferventcoder

I know this is old now, but still. Perhaps, checking if the extraction directory isn't inside the Get-BinDir/Get-ToolsDir or the package Directory or temporary directory would be enough to flag this?

Could cause a false positive for extensions/plugins that need to be extracted to another program install directory though. (So maybe change to a note instead of guideline?)

AdmiringWorm avatar Mar 05 '17 04:03 AdmiringWorm