Skip to content

Add hoplimit argument to override default hopLimit in config#745

Draft
ppicazo wants to merge 2 commits into
meshtastic:masterfrom
ppicazo:add-hoplimit-argument
Draft

Add hoplimit argument to override default hopLimit in config#745
ppicazo wants to merge 2 commits into
meshtastic:masterfrom
ppicazo:add-hoplimit-argument

Conversation

@ppicazo

@ppicazo ppicazo commented Feb 26, 2025

Copy link
Copy Markdown
Contributor

Add command line argument for hopLimit to override the default one set in the config for --sendtext, --traceroute, --request-telemetry, and --request-position.

meshtastic --host 192.168.1.10 --traceroute df81f5d7 --hoplimit 5
meshtastic --host 192.168.1.10 --sendtext "hello world" --hoplimit 2

If hoplimit is not specified, defaults to the hoplimit in set in the radio config.

Add command line argument for hopLimit to override the default one set in the config for `--sendtext`, `--traceroute`, `--request-telemetry`, and `--request-position`.
@ppicazo

ppicazo commented Feb 26, 2025

Copy link
Copy Markdown
Contributor Author

@ianmcorvidae could you take a look at this?

@ppicazo ppicazo marked this pull request as draft February 26, 2025 23:42
@ppicazo

ppicazo commented Feb 26, 2025

Copy link
Copy Markdown
Contributor Author

Making some more changes...

@rc14193

rc14193 commented Mar 15, 2025

Copy link
Copy Markdown
Contributor

Can you add the hopLimit parameter to sendWaypoint and sendAlert since you already address the other two funcs that need it added according to this issue

@ianmcorvidae

Copy link
Copy Markdown
Contributor

Hi; sorry it took me quite a while to look into this. Your changes look good to me, so if you're interested in putting it out of draft I think it could be merged.

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

Successfully merging this pull request may close these issues.

3 participants