Skip to content
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

(GH-121) Capture And Remove Files Related to Package Installation #270

Merged

Conversation

ferventcoder
Copy link
Member

  • Files snapshot
  • Capture files snapshot at end of install/upgrade
  • Remove unchanged files on uninstall
  • Backup changed files on upgrade

Closes #121

Define the package files classes and the package information
file they will produce.
Add ability to read and save package file information.
Add the interaction to the package information service to read the
files file and save to the files file.
Allow IFileSystem to read a file and return the file bytes as an array.
To compute hashes in files, add a hash provider that can determine
hashes of files.
Capture the package hashes after everything, including the
install script, has had a chance to run.
- Rename Md5HashProvider to CryptoHashProvider and provide enumeration
   for HashAlgorithm Crytpo providers.
- Implement adapter for HashAlgorithm.
- Update specs for CryptoHashProvider
- Specs for FilesService
@ferventcoder ferventcoder self-assigned this May 4, 2015
@ferventcoder ferventcoder changed the title (GH-121) Capture And Remove Files Related to Package Installation [DO NOT MERGE](GH-121) Capture And Remove Files Related to Package Installation May 4, 2015
When the files that were installed during install are unchanged, they
should be removed from the package directory. Any files that have
changed should stick around.
To see messages coming from Nuget, prefix it with "[NuGet]" so that
there is no confusion where the messages are coming from.
While adding scenarios it was discovered that the empty directories
were not being removed, so the service will ensure those are removed
upon removing a package with no changed files in it.
Results is actually a singular result, so the naming is off. Fix that
to be appropriate.
The entire package checksum is not needed at this time so remove it.
When files have changed in a package, back those files up prior to
upgrade.
Add fault tolerance around deleting the nupkg with an error. Add fault
tolerance around deleting the rest of the files on the system.
@ferventcoder ferventcoder changed the title [DO NOT MERGE](GH-121) Capture And Remove Files Related to Package Installation (GH-121) Capture And Remove Files Related to Package Installation May 15, 2015
ferventcoder added a commit that referenced this pull request May 15, 2015
…napshot

(GH-121) Capture And Remove Files Related to Package Installation
@ferventcoder ferventcoder merged commit 3bb8421 into chocolatey:stable May 15, 2015
@ferventcoder ferventcoder deleted the ticket/stable/GH-121-file-snapshot branch May 15, 2015 13:36
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.

1 participant