Feature #2805
closedEmbed dependency installation in Jenkins-CI scripts
100%
Description
CI scripts should automatically install necessary dependencies / update packages to the latest versions.
Updated by Junxiao Shi almost 10 years ago
- Tracker changed from Task to Feature
- Category changed from Tests to Build
Updated by Alex Afanasyev almost 10 years ago
Why build category? This is clearly for tests. If you consider "tests" to refer exclusively to integration tests, then I would rename the category and add another tests category to refer to things related to unit testing.
Updated by Junxiao Shi almost 10 years ago
ndn-cxx "Tests" contains both unit tests and integ.
This issue is about build script so it's Build category.
Updated by Alex Afanasyev almost 10 years ago
It is a build script for testing environment, not general build script. In my mind, build only refers to general build elements: waf, wscript, and all related tools.
Updated by Junxiao Shi almost 10 years ago
Both waf and CI are under Build category; it's unnecessary to distinguish these two.
Look at NFD's Build category: it even contains Jenkins slave deployment.
Further discussion on this topic should move to nfd-dev mailing list.
Updated by Alex Afanasyev almost 10 years ago
This discussion is too minor to go on the mailing list, and I don't really care much, it wasn't just too straightforward for me.
Updated by Junxiao Shi almost 10 years ago
- Related to Feature #2843: Embed dependency installation in Jenkins-CI scripts added
Updated by Joao Pereira almost 10 years ago
Isn't this issue solved by copying the 00-deps.sh script from ndn-tools?
Updated by Junxiao Shi almost 10 years ago
No. ndn-tools has more dependencies than ndn-cxx.
Updated by Joao Pereira almost 10 years ago
Maybe my question was not well formed :D
What I was asking was if the only thing that is needed in this Issue is to add a file similar to the one we have in ndn-tools, but with the correct dependencies, or not.
Updated by Alex Afanasyev almost 10 years ago
- Status changed from New to Code review
- Assignee set to Alex Afanasyev
- Target version set to v0.4
- % Done changed from 0 to 100
Updated by Alex Afanasyev almost 10 years ago
Updated by Yingdi Yu over 9 years ago
- Blocks Feature #3007: OpenSSL dependency added
Updated by Junxiao Shi over 9 years ago
- Blocks deleted (Feature #3007: OpenSSL dependency)
Updated by Junxiao Shi over 9 years ago
- Status changed from Code review to Closed