Named Data Networking (NDN) Tools #26923

MostAwesomeDude · 2017-06-28T07:17:06Z

Motivation for this change

I want to play with NDN.

Things done

Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)
- NixOS
- macOS
- Linux
Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
Tested execution of all binary files (usually in ./result/bin/)
Fits CONTRIBUTING.md.

I had to use versions from git because this stuff is so bleeding-edge.

mention-bot · 2017-06-28T07:17:07Z

@MostAwesomeDude, thanks for your PR! By analyzing the history of the files in this pull request, we identified @dezgeg and @sjmackenzie to be potential reviewers.

MostAwesomeDude · 2017-06-28T07:18:54Z

Hm. These actually do have releases; I'm going to fix that and rebase.

0xABAB · 2017-06-28T14:30:11Z

pkgs/development/libraries/ndn-cxx/default.nix

+  # To do battle with the tests, add --with-tests to the configure above. ~ C.
+  doCheck = false;
+  checkPhase = ''
+    LDD_LIBRARY_PATH=build/ build/unit-tests


Don't you mean LD_LIBRARY_PATH?

Yes. Fixed. The tests still don't pass, of course.

0xABAB · 2017-06-28T14:31:37Z

pkgs/servers/named-data/nfd/default.nix

+  preConfigure = ''
+    patchShebangs waf
+    ./waf configure \
+      --boost-includes=${boost.dev}/include \


Needs quoting.

Just this one, or in all of the configures?

All locations.

0xABAB

Minor stuff. Thanks for your contribution.

Mic92 · 2017-06-29T07:55:31Z

pkgs/tools/networking/named-data/repo-ng/default.nix

+  version = "2b7b83";
+in
+stdenv.mkDerivation {
+  name = "ndn-tools-0.1-${version}";


name is probably wrong.

What a mistake! Thanks.

MostAwesomeDude · 2017-06-30T18:12:54Z

I've addressed all feedback and am ready for merge. nfd works, at least. I haven't gotten far enough to test the "ndnsec" toolset. repo-ng is cantankerous, which is unsurprising since it is still immature.

matthewbauer

Fix merge conflicts

mmahut · 2019-08-13T08:13:26Z

Are there any updates on this pull request, please?

MostAwesomeDude · 2019-09-01T15:05:33Z

Hi, I rebased the branch and also bumped the versions of all new packages. It seems that, at some point, a derivation based on my ndn-cxx work got merged, so now the branch's history is a bit messy. I can redo all of the commits, if desired. I haven't been poking at this stuff for a while, though.

sjmackenzie · 2019-09-07T17:51:21Z

@matthewbauer I believe the merge conflicts were resolved. Can you merge please?

Mic92 · 2019-09-11T10:03:06Z

pkgs/servers/named-data/nfd/default.nix

+  };
+  buildInputs = [ wafHook
+                  libpcap doxygen boost pkgconfig python pythonPackages.sphinx
+                  openssl ndn-cxx ];


nativeBuildInputs = [ wafHook doxygen pkgconfig python pythonPackages.sphinx ];

Mic92 · 2019-09-11T10:03:55Z

pkgs/tools/networking/named-data/ndn-tools/default.nix

+    sha256 = "1sjhyhc9ww38vyf3qmjzh5c4c0jc43ypzjmpqwsbq8y5zzln0gfx";
+  };
+
+  buildInputs = [ wafHook libpcap openssl boost pkgconfig python pythonPackages.sphinx git ndn-cxx ];


nativeBuildInputs = [ wafHook pkgconfig python pythonPackages.sphinx git ];

btw. why is git required here?

Mic92 · 2019-09-11T10:04:35Z

pkgs/tools/networking/named-data/repo-ng/default.nix

+    sha256 = "0pdinvq3cf458bmm2mxdlidrmmr38in1563ra8q0ch3wp45va2di";
+  };
+  buildInputs = [ wafHook libpcap openssl doxygen boost sqlite pkgconfig
+                  python pythonPackages.sphinx ndn-cxx ];


nativeBuildInputs = [ wafHook doxygen pkgconfig python pythonPackages.sphinx ];

Mic92 · 2019-09-11T10:05:15Z

pkgs/development/libraries/ndn-cxx/default.nix

  };
+
  nativeBuildInputs = [ pkgconfig wafHook ];
  buildInputs = [ openssl doxygen boost sqlite python pythonPackages.sphinx];


also doxygen, python and sphinx should be in nativeBuildInputs

And use fetchFromGitHub.

ndn-tools: use fetchFromGitHub

repo-ng: Fix up name and pick a working revision. And use fetchFromGitHub too.

Also bump nfd, ndn-tools, repo-ng, as they all seem very tied-together in terms of versioning.

roberth · 2020-11-10T17:09:35Z

ndn-cxx has been updated to 0.7.1 in master. You could rebase -i and omit the update, then add yourself to the maintainers in a new commit.

stale · 2021-07-21T01:12:12Z

I marked this as stale due to inactivity. → More info

0xABAB reviewed Jun 28, 2017

View reviewed changes

0xABAB suggested changes Jun 28, 2017

View reviewed changes

MostAwesomeDude force-pushed the ndn branch from 826eac3 to e308994 Compare June 28, 2017 16:52

Mic92 reviewed Jun 29, 2017

View reviewed changes

0xABAB approved these changes Jun 30, 2017

View reviewed changes

matthewbauer requested changes Apr 21, 2018

View reviewed changes

veprbl added the 2.status: merge conflict label Jan 9, 2019

MostAwesomeDude force-pushed the ndn branch from 6531803 to dd712eb Compare September 1, 2019 15:04

ofborg bot added 8.has: package (new) 11.by: package-maintainer 10.rebuild-darwin: 1-10 10.rebuild-linux: 1-10 and removed 2.status: merge conflict labels Sep 1, 2019

Mic92 reviewed Sep 11, 2019

View reviewed changes

lheckemann added this to the 20.03 milestone Sep 12, 2019

disassembler modified the milestones: 20.03, 20.09 Feb 5, 2020

MostAwesomeDude added 3 commits May 26, 2020 00:48

nfd: init at 0.5.1

eb24691

And use fetchFromGitHub.

ndn-tools: init at 0.4

b4027ab

ndn-tools: use fetchFromGitHub

ndn-repo-ng: init at 05zv7i

3cea5a4

repo-ng: Fix up name and pick a working revision. And use fetchFromGitHub too.

MostAwesomeDude added 5 commits May 26, 2020 00:48

ndn-cxx, nfd, ndn-tools, repo-ng: Fix up quotes in configurePhase.

a2b5c09

ndn: Fix wrong outputs.

9d8c511

ndn-cxx: 0.5.1 -> 0.6.6

ae2cd4b

Also bump nfd, ndn-tools, repo-ng, as they all seem very tied-together in terms of versioning.

all-packages: Fix mergo.

3b082e9

ndn: Address review feedback.

884593b

MostAwesomeDude force-pushed the ndn branch from d0e746a to 884593b Compare May 26, 2020 08:28

MostAwesomeDude requested a review from matthewbauer May 26, 2020 08:28

FRidh modified the milestones: 20.09, 21.03 Dec 20, 2020

SuperSandro2000 added the 2.status: merge conflict label Jan 18, 2021

stale bot added the 2.status: stale label Jul 21, 2021

MostAwesomeDude closed this Jul 29, 2021

MostAwesomeDude deleted the ndn branch July 29, 2021 07:15

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Named Data Networking (NDN) Tools #26923

Named Data Networking (NDN) Tools #26923

MostAwesomeDude commented Jun 28, 2017

mention-bot commented Jun 28, 2017

MostAwesomeDude commented Jun 28, 2017

0xABAB Jun 28, 2017

MostAwesomeDude Jun 28, 2017

0xABAB Jun 28, 2017

MostAwesomeDude Jun 28, 2017

0xABAB Jun 28, 2017

0xABAB left a comment

Mic92 Jun 29, 2017

MostAwesomeDude Jun 29, 2017

MostAwesomeDude commented Jun 30, 2017

matthewbauer left a comment

mmahut commented Aug 13, 2019

MostAwesomeDude commented Sep 1, 2019

sjmackenzie commented Sep 7, 2019

Mic92 Sep 11, 2019

Mic92 Sep 11, 2019

Mic92 Sep 11, 2019

Mic92 Sep 11, 2019

roberth commented Nov 10, 2020

stale bot commented Jul 21, 2021

Named Data Networking (NDN) Tools #26923

Named Data Networking (NDN) Tools #26923

Conversation

MostAwesomeDude commented Jun 28, 2017

Motivation for this change

Things done

mention-bot commented Jun 28, 2017

MostAwesomeDude commented Jun 28, 2017

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

0xABAB left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

MostAwesomeDude commented Jun 30, 2017

matthewbauer left a comment

Choose a reason for hiding this comment

mmahut commented Aug 13, 2019

MostAwesomeDude commented Sep 1, 2019

sjmackenzie commented Sep 7, 2019

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

roberth commented Nov 10, 2020

stale bot commented Jul 21, 2021