search

sns3 / sns3-satellite

Satellite module for ns-3 simulator
https://www.sns3.org
GNU General Public License v3.0
141 stars 58 forks source link

Installation problem #54

Open P1mentaX opened 2 months ago

P1mentaX commented 2 months ago

When I tried to download sns3, I encountered problems: I installed ns3.37 as the master version recommends, but it returned the error shown in image . Captura de tela 2024-08-29 155321

I also tried downloading the dev version for 3.42, and it returned the errors shown in the other images.

Captura de tela 2024-08-29 154657 Captura de tela 2024-08-29 154730

Can someone help me with this problem?

mruizfda commented 2 months ago

same problem witn ns-3.37, any solution please?

Ssh1y commented 1 month ago

same problem witn ns-3.42, any solution please?

bastientauran commented 1 week ago

Hello, You need to make sure that the 3 modules (satellite, traffic and magister-stats) are coherent with the ns3 version. For ns-3.37, you need:

thebear132 commented 1 week ago

Hello, You need to make sure that the 3 modules (satellite, traffic and magister-stats) are coherent with the ns3 version. For ns-3.37, you need:

* satellite on commit [04dc901](https://github.com/sns3/sns3-satellite/commit/04dc901404beb9b320cd018f4a8d6438f402ad83)

* traffic on commit c68daaf1

* magister-stats on commit c613ba7c
  For ns-3.42, you need:

* satellite on commit [19779c2](https://github.com/sns3/sns3-satellite/commit/19779c2aae6534ff6dbd5a75ec3a901d49e75ccf)

* traffic on commit d9aa8ce4

* magister-stats on commit 95ae2a8f
  From ns-3.41, you can use tags for all 3 modules to help getting the correct version

Hi, For ns-3.42 this results in errors. For the satellite repo i did $ git checkout 19779c2aae6534, while the two others are up to date when doing $git log in their respective folders: bear@debian:~/Desktop/ns-allinone-3.42/ns-3.42/contrib$ ls --> magister-stats satellite traffic

I have followed the CMake installation, where the errors happen when doing ./ns3 build Screenshot_20241106_154245

bastientauran commented 1 week ago

Hello thebear132, It seems that some dependencies may be missing for some files (map in your case). Depending on the compilation order of source files, you may this issue. I will perform a full analysis of all classes to add missing includes. However, it will only be done on a new commit from dev branch. For you commit, i cannot edit it. Thus, either you can switch to my latest commit once it is pushed, or you can add manually missing includes following the notes from your compiler.

thebear132 commented 1 week ago

A temporary fix is to use a certain ns-3 version and change the 3 repositories to these certain commits

But again, this is temporary. Looking forward to your fix for ns-3.42 @bastientauran

thebear132 commented 1 week ago

Hello thebear132, It seems that some dependencies may be missing for some files (map in your case). Depending on the compilation order of source files, you may this issue. I will perform a full analysis of all classes to add missing includes. However, it will only be done on a new commit from dev branch. For you commit, i cannot edit it. Thus, either you can switch to my latest commit once it is pushed, or you can add manually missing includes following the notes from your compiler.

Thank you for looking into this issue. When do you expect that your bugfix will be available?

bastientauran commented 1 week ago

The 3 repositories have just been pushed:

thebear132 commented 1 week ago

The 3 repositories have just been pushed:

* satellite: dev branch, commit [00f4641](https://github.com/sns3/sns3-satellite/commit/00f46416850cc8de2be6321816924cda7df0a546)

* traffic: master branch, commit 76c9318

* magister-stats: master branch, commit b984bce
  The latest version of these 3 modules should still work with ns-3.42, in addition to ns-3.43

These changes almost work. All they need is one include and a '&' added to one of the files. @juhlemanden14 and I have made a pull request which you can merge into the dev branch https://github.com/sns3/sns3-satellite/pull/55.

bastientauran commented 1 week ago

Thank you for the feedback and the PR. This repository is only a mirror of a private gitlab one, thus I pushed your edits on gitlab and they have been mirrored on commit 202c72ed. If everything is good, we can close the PR. For information, on which distribution do you compile and run SNS3 ?

thebear132 commented 1 week ago

Thank you for your help. You can close the Pull-request if you want to. We are running Linux Debian 6.1.0-26-amd64