jyapayne/nimble
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge pull request #448 from genotrance/#280

Browse source 
Fix for #280
This commit is contained in:
Dominik Picheta 2018-01-08 13:20:49 +00:00 • committed by GitHub
commit c8cd1d9286
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 10 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

10
src/nimble.nim
View file

@ -468,7 +468,17 @@ proc install(packages: seq[PkgTuple],
let (downloadDir, downloadVersion) = let (downloadDir, downloadVersion) =
downloadPkg(url, pv.ver, meth, options) downloadPkg(url, pv.ver, meth, options)
try: try:
# Run pre-install hook in download directory now that package is downloaded
cd downloadDir:
if not execHook(options, true):
raise newException(NimbleError, "Pre-hook prevented further execution.")
result = installFromDir(downloadDir, pv.ver, options, url) result = installFromDir(downloadDir, pv.ver, options, url)
# Run post-install hook in installed directory now that package is installed
# Standard hooks run in current directory so it won't detect this new package
cd result.pkg.myPath.parentDir():
discard execHook(options, false)
except BuildFailed: except BuildFailed:
# The package failed to build. # The package failed to build.
# Check if we tried building a tagged version of the package. # Check if we tried building a tagged version of the package.