-
Notifications
You must be signed in to change notification settings - Fork 22
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
First try for a pull request #4
base: master
Are you sure you want to change the base?
Conversation
Changes: 1/- setup.py is modified 2/- new directory "ccnxmn" is created with 4 new files: - __init__.py - CCN_cli.py - CCN_node.py - conf_parser.py 3/- bin/miniccnx is changed -from mininet.cli import CLI -from mininet.node import CPULimitedCCNHost, CCNHost +from ccnxmn.CCN_cli import CCNxCLI +from ccnxmn.CCN_node import CPULimitedCCNHost, CCNHost -from mininet.conf_parser import parse_hosts,parse_routers, parse_links +from ccnxmn.conf_parser import parse_hosts,parse_routers, parse_links - CLI(net) + CCNxCLI(net)
HI,
That is very simple test, but it does not work. Did i miss any key steps? PS: The vlc has installed the plugin of ccnx. Regards, |
Hi, Thank you for sharing your test. I myself would like to learn about CCNx. Have you tried with the original miniccnx? Regards, PhiHo On Tue, May 20, 2014 at 10:18 PM, IanFuxing notifications@github.comwrote:
|
P.S: Please give me details of the environment (OS, version...) and what On Wed, May 21, 2014 at 4:36 AM, Phiho Hoang hohoangphi@gmail.com wrote:
|
Taking the hint from Fuxing Ian Chen, CCNxMininet class was created in ccnxmn/CCN_net.py. If there are errors, please quote the errors, thanks. |
Hi IanFuxing, I've just committed some changes. Please try again and please quote the error messages. I will try to setup the environment to reproduce the errors. Regards, PhiHo On Wed, May 21, 2014 at 5:58 AM, Phiho Hoang hohoangphi@gmail.com wrote:
|
Received. Give us a few days to try it out. Many thanks! Hi Christian, This is my first pull request. I hope you can make use of it. Cheers, PhiHoYou can merge this Pull Request by running git pull https://github.com/Phi-Ho/mn-ccnx master Or view, comment on, or merge it at: #4
File Changes
Patch Links:
— |
Hi Christian,
This is my first pull request.
I hope you can make use of it.
Cheers,
PhiHo