Skip to content
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

Network Interface agnostic make process #29

Open
D3vil0p3r opened this issue Oct 3, 2024 · 7 comments
Open

Network Interface agnostic make process #29

D3vil0p3r opened this issue Oct 3, 2024 · 7 comments

Comments

@D3vil0p3r
Copy link

Hey @davidmcgrew I'm trying to package mercury for some Linux distros. What I see as constraint is the need of specifying the network interface when running make install by specifying it inside the mercury.cfg file.

Is it possible to make mercury agnostic of network interface during the build and install time and allow to specify it after the installation (so not making it tied to make install command)?

@davidmcgrew
Copy link
Member

davidmcgrew commented Oct 3, 2024 via email

@D3vil0p3r
Copy link
Author

D3vil0p3r commented Oct 3, 2024

I noted also that, if make install is provided without config file, it produces an error. So by your solution, we should be able to run make install also with no providing any config file.

@noraj what do you think about the proposed solution?

@noraj
Copy link

noraj commented Oct 3, 2024

For being easily package on any Linux distro, the program should not require knowing an interface name at compile / build time nor package time. Anyway compilation and packaging will not occur on the same machine as the one where the package will be installed. It should look for the interface in the configuration file only at runtime.

@D3vil0p3r
Copy link
Author

@davidmcgrew did you give a look into it? Can you manage the recon of network interface directly at code level or by specifying it by an argument (i.e., mercury -I/--interface eth0)?

@davidmcgrew
Copy link
Member

This change should appear in the next release today or tomorrow. Apologies for the delay. Please let me know if there are any issues with the changes.

@D3vil0p3r
Copy link
Author

D3vil0p3r commented Dec 21, 2024

Hey @davidmcgrew I just saw you pushed the new release of mercury and I see the commit where you removed default interface from the build process. Now the interface will be specified at runtime by an argument like -i/--interface? If so, can you please update README.md as well? If not, will it be still specified in the mercury.cfg config file and only checked at runtime?

@andrewchi
Copy link
Collaborator

andrewchi commented Dec 23, 2024

@D3vil0p3r Thanks for the reminder to update the documentation; apologies that we missed that. Look for it the first full week of January when folks return from holiday.

The interface can be specified either by command line argument ([-c or --capture] capture_interface) or an entry in mercury.cfg. There simply won't be one placed in a config file by default.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants