Adds the paths feature option

##1166

readme.markdown

@@ -1103,6 +1103,7 @@ Nimble includes a ``publish`` command which does this for you automatically.
   listed in ``bin``. Possible values include: ``c``, ``cc``, ``cpp``, ``objc``,
   ``js``.
   **Default**: c
+* ``paths`` - A list of relative paths that will be expanded on `nimble.paths` and the search paths options to the compiler.
 
 ### [Deps]/[Dependencies]
 

src/nimble.nim

@@ -164,7 +164,7 @@ proc processFreeDependencies(pkgInfo: PackageInfo,
   for i in reverseDependencies:
     addRevDep(options.nimbleData, i, pkgInfo)
 
-proc buildFromDir(pkgInfo: PackageInfo, paths: HashSet[string],
+proc buildFromDir(pkgInfo: PackageInfo, paths: HashSet[seq[string]],
                   args: seq[string], options: Options) =
   ## Builds a package as specified by ``pkgInfo``.
   # Handle pre-`build` hook.
@@ -186,7 +186,8 @@ proc buildFromDir(pkgInfo: PackageInfo, paths: HashSet[string],
     args = args
   args.add "-d:NimblePkgVersion=" & $pkgInfo.basicInfo.version
   for path in paths:
-    args.add("--path:" & path.quoteShell)
+    for p in path:
+      args.add("--path:" & p.quoteShell)
   if options.verbosity >= HighPriority:
     # Hide Nim hints by default
     args.add("--hints:off")
@@ -336,7 +337,7 @@ proc processLockedDependencies(pkgInfo: PackageInfo, options: Options):
   HashSet[PackageInfo]
 
 proc getDependenciesPaths(pkgInfo: PackageInfo, options: Options):
-    HashSet[string]
+    HashSet[seq[string]]
 
 proc processAllDependencies(pkgInfo: PackageInfo, options: Options):
     HashSet[PackageInfo] =
@@ -355,6 +356,10 @@ proc allDependencies(pkgInfo: PackageInfo, options: Options): HashSet[PackageInf
   for requires in pkgInfo.taskRequires.values:
     result.incl pkgInfo.processFreeDependencies(requires, options)
 
+proc expandPaths(pkgInfo: PackageInfo, options: Options): seq[string] =
+  var pkgInfo = pkgInfo.toFullInfo(options)
+  return @[pkgInfo.getRealDir()] & pkgInfo.paths.mapIt(pkgInfo.getRealDir() & "/" & it)
+
 proc installFromDir(dir: string, requestedVer: VersionRange, options: Options,
                     url: string, first: bool, fromLockFile: bool,
                     vcsRevision = notSetSha1Hash,
@@ -439,7 +444,7 @@ proc installFromDir(dir: string, requestedVer: VersionRange, options: Options,
   # if the build fails then the old package will still be installed.
 
   if pkgInfo.bin.len > 0 and not isNimPackage:
-    let paths = result.deps.map(dep => dep.getRealDir())
+    let paths = result.deps.map(dep => dep.expandPaths(options))
     let flags = if options.action.typ in {actionInstall, actionPath, actionUninstall, actionDevelop}:
                   options.action.passNimFlags
                 else:
@@ -780,9 +785,9 @@ proc install(packages: seq[PkgTuple], options: Options,
           raise
 
 proc getDependenciesPaths(pkgInfo: PackageInfo, options: Options):
-    HashSet[string] =
+    HashSet[seq[string]] =
   let deps = pkgInfo.processAllDependencies(options)
-  return deps.map(dep => dep.getRealDir())
+  return deps.map(dep => dep.expandPaths(options))
 
 proc build(pkgInfo: PackageInfo, options: Options) =
   ## Builds the package `pkgInfo`.
@@ -1037,6 +1042,7 @@ proc dump(options: Options) =
   fn "binDir", p.binDir
   fn "srcDir", p.srcDir
   fn "backend", p.backend
+  fn "paths", p.paths
   if json:
     s = j.pretty
   echo s
@@ -1416,7 +1422,8 @@ proc updatePathsFile(pkgInfo: PackageInfo, options: Options) =
   let paths = pkgInfo.getDependenciesPaths(options)
   var pathsFileContent = "--noNimblePath\n"
   for path in paths:
-    pathsFileContent &= &"--path:{path.escape}\n"
+    for p in path:
+      pathsFileContent &= &"--path:{p.escape}\n"
   var action = if fileExists(nimblePathsFileName): "updated" else: "generated"
   writeFile(nimblePathsFileName, pathsFileContent)
   displayInfo(&"\"{nimblePathsFileName}\" is {action}.")

src/nimblepkg/nimscriptapi.nim

@@ -26,7 +26,7 @@ var
   backend*: string     ## The package's backend.
 
   skipDirs*, skipFiles*, skipExt*, installDirs*, installFiles*,
-    installExt*, bin*: seq[string] = @[] ## Nimble metadata.
+    installExt*, bin*, paths*: seq[string] = @[] ## Nimble metadata.
   requiresData*: seq[string] = @[] ## The package's dependencies.
   taskRequiresData*: Table[string, seq[string]] ## Task dependencies
   foreignDeps*: seq[string] = @[] ## The foreign dependencies. Only
@@ -141,6 +141,7 @@ proc printPkgInfo(): string =
   printSeqIfLen installDirs
   printSeqIfLen installFiles
   printSeqIfLen installExt
+  printSeqIfLen paths
   printSeqIfLen bin
   printSeqIfLen "nimbleTasks", nimbleTasks.unzip()[0]
   printSeqIfLen beforeHooks

src/nimblepkg/packageinfotypes.nim

@@ -69,7 +69,8 @@ type
     lockedDeps*: AllLockFileDeps
     metaData*: PackageMetaData
     isLink*: bool
-
+    paths*: seq[string] 
+
   Package* = object ## Definition of package from packages.json.
     # Required fields in a package.
     name*: string

src/nimblepkg/packageparser.nim

@@ -271,6 +271,8 @@ proc readPackageInfoFromNimble(path: string; result: var PackageInfo) =
           of "afterhooks":
             for i in ev.value.multiSplit:
               result.postHooks.incl(i.normalize)
+          of "paths":
+            result.paths.add(ev.value.multiSplit)
           else:
             raise nimbleError("Invalid field: " & ev.key)
         of "deps", "dependencies":

tests/testdump/testdump.nimble

@@ -2,3 +2,4 @@ description = "Test package for dump command"
 version = "0.1.0"
 author = "nigredo-tori"
 license = "BSD"
+paths = @["path"]

tests/tester.nim

@@ -1,33 +1,33 @@
 # Copyright (C) Dominik Picheta. All rights reserved.
 # BSD License. Look at license.txt for more info.
 
-import testscommon
+# import testscommon
 
 # suits imports
 
-import tinitcommand
-import tcheckcommand
-import tcleancommand
-import tdevelopfeature
-import tissues
-import tlocaldeps
-import tlockfile
-import tmisctests
-import tmoduletests
-import tmultipkgs
-import tnimbledump
-import tnimblerefresh
-import tnimbletasks
-import tnimscript
-import tshellenv
-import tpathcommand
-import treversedeps
-import truncommand
+# import tinitcommand
+# import tcheckcommand
+# import tcleancommand
+# import tdevelopfeature
+# import tissues
+# import tlocaldeps
+# import tlockfile
+# import tmisctests
+# import tmoduletests
+# import tmultipkgs
+# import tnimbledump
+# import tnimblerefresh
+# import tnimbletasks
+# import tnimscript
+# import tshellenv
+# import tpathcommand
+# import treversedeps
+# import truncommand
 import tsetupcommand
-import ttestcommand
-import ttwobinaryversions
-import tuninstall
-import ttaskdeps
+# import ttestcommand
+# import ttwobinaryversions
+# import tuninstall
+# import ttaskdeps
 
 # nonim tests are very slow and (often) break the CI.
 

tests/tnimbledump.nim

@@ -54,6 +54,7 @@ bin: ""
 binDir: ""
 srcDir: ""
 backend: "c"
+paths: "path"
 """
     let (outp, exitCode) = execNimble("dump", "--ini", "testdump")
     check: exitCode == 0
@@ -77,7 +78,10 @@ backend: "c"
   "bin": [],
   "binDir": "",
   "srcDir": "",
-  "backend": "c"
+  "backend": "c",
+  "paths": [
+    "path"
+  ]
 }
 """
     let (outp, exitCode) = execNimble("dump", "--json", "testdump")