krnl386,
@krnl386@lemmy.ca avatar

Good old git blame lol! Not only can you determine when the change was made and where, it’s trivial to look up the author of the commit: github.com/…/562e0d570d93cfcfdebab1215a2f04efa64a…

To be fair, the author’s first language may not be English…

Is anyone interested in submitting a pull request? Looks like Github contributions are accepted.

TheGingerNut,
@TheGingerNut@lemmy.blahaj.zone avatar

My dude. It’s not a bug, OP just used small i instead of capital. Please don’t bother developers over this.

krnl386,
@krnl386@lemmy.ca avatar

I think descriptive and useful error messages are OK to report as enhancements. They don’t have to be functional bugs.

TheGingerNut,
@TheGingerNut@lemmy.blahaj.zone avatar

There might be a reason it was never fixed. Something stupid like a lot of 30 year old scripts relying on this out come. Hope not, you’re right that error message is ass

0xb,
@0xb@lemmy.world avatar

-Me talking to people on twitter after nov 22 and before leaving

0x0,

Wow, that is an unhelpful error message. It could have told you it was expecting a number. It turns out that -i is short for “interval” and expects a number, whereas -I is used to specify an interface.

aniki,

This exactly. And because this is open-source, its super easy to track this down. I searched for the source of the error message:

github.com/iputils/iputils/blob/…/ping.c#L237

see this is ping_strtod function - so search for that function name and the first hit is the ‘-i’ case:


<span style="color:#323232;">		case 'i':
</span><span style="color:#323232;">		{
</span><span style="color:#323232;">			double optval;
</span><span style="color:#323232;">
</span><span style="color:#323232;">			optval = ping_strtod(optarg, _("bad timing interval"));
</span><span style="color:#323232;">			if (isless(optval, 0) || isgreater(optval, (double)INT_MAX / 1000))
</span><span style="color:#323232;">				error(2, 0, _("bad timing interval: %s"), optarg);
</span><span style="color:#323232;">			rts.interval = (int)(optval * 1000);
</span><span style="color:#323232;">			rts.opt_interval = 1;
</span><span style="color:#323232;">		}
</span><span style="color:#323232;">			break;
</span>
Sylver,

Wait a second… That’s MY ip!

herrcaptain,

Wait … the ping is coming from inside the house!

0x4E4F,

Holy shit 😱 (unplugs everything from mains… waits in terror in a corner…)

popekingjoe,
@popekingjoe@lemmy.world avatar

Same tbh.

bleistift2,

It’s actually a technical term in language theory – a term that cannot be produced by the language, but is still considered part of the ‘universe’ (i.e., the set of all terms) is called ‘garbage’ or ‘junk’.

Since I can’t find a source online to verify this claim, this might just have been the case in my courses…

damium,

“Invalid” or “unparseable” are more understandable descriptors in normal language. I don’t think I ever heard of garbage/junk being used for that in language theory but it may be domain specific usage.

felbane,

See, even ping hates “consistent device naming!”

NotMelon,

Hahahahah you just doxed yourself hahahahah

Rooki,
@Rooki@lemmy.world avatar

Ahh yes get doxed through local ip XD

Thade780,

Woooooosh

AceQuorthon,

I got your IP adress, see you soon kiddo 😎

roguetrick,

I'm already in the network... Wait...

Bezier,
@Bezier@suppo.fi avatar

OP must be my neighbour

  • All
  • Subscribed
  • Moderated
  • Favorites
  • linuxmemes@lemmy.world
  • DreamBathrooms
  • magazineikmin
  • Durango
  • Youngstown
  • vwfavf
  • slotface
  • ngwrru68w68
  • khanakhh
  • rosin
  • kavyap
  • thenastyranch
  • PowerRangers
  • mdbf
  • ethstaker
  • anitta
  • hgfsjryuu7
  • osvaldo12
  • cubers
  • GTA5RPClips
  • modclub
  • InstantRegret
  • everett
  • tacticalgear
  • normalnudes
  • tester
  • cisconetworking
  • Leos
  • provamag3
  • All magazines