I fixed a bug in Python

http://nedbatchelder.com/blog/201211/i_fixed_python.html

by llambda • 14 years ago

131 23 14 years ago

secure• 14 years ago

link bait. actual title: i fixed an obscure bug in the cpython tokenizer.

I apologize for the "link bait." I didn't intend it as such, I was actually making fun of my "accomplishment," having contributed a tiny patch that fixed an obscure problem.

I can see how it reads very differently on Hacker News...

smoyer• 14 years ago

Sorry for the negative comment (I'm about to delete). I missed the tongue-in-cheek nature of your posting and as others pointed out. It was a real bug.

gruseom• 14 years ago

You have nothing to apologize for!

nmb• 14 years ago

meh, forget the haters. congrats on your bugfix, I'd be pretty excited about contributing to CPython, too!

JulianWasTaken• 14 years ago

FWIW Ned, I had to update my (generally lame) résumé recently and was a bit embarrassed to put CPython on my list of projects that I've contributed to due to the same type of thing: it was a small patch that fixed an obscure issue.

Anyways, I did it anyhow (with the caveat to make me feel better :P) but I felt the same way about the "accomplishment" :). So congrats!

(This is tos9 by the way).

scott_s• 14 years ago

Worry not, your title is fine.

antrod• 14 years ago

Screw it— you have nothing to apologize for. It is because of people like you— fixing bugs big an small— that Python is a viable for big projects.

And to the dude who put it on his resume— good move. This helps a ton when someone is screening resumes.

kelnos• 14 years ago

Don't be silly. I have 2 super-trivial patches in the Linux kernel tree (one to fix an array-indexing bug, and another to blacklist a feature on a piece of broken hardware). They're simple patches, but I'm nonetheless really happy to have contributed to such a giant project. You shouldn't apologize for feeling the same and wanting to post to HN about it!

manojlds• 14 years ago

Which is what you would expect, unless the OP was Guido, wouldn't you?

If my patch, how much so ever minuscule, was merged into CPython, I would be similarly elated as well.

memset• 14 years ago

This is great! It's so hard to find bugs in software tools because the conventional wisdom is that "it's never a bug in the compiler."

Interesting that you had to create two patches for python 2.x and 3.x. (Frustrating, no?)

Seems like you were able to get this submitted and fixed fairly quickly; the Python guys seem like a welcoming group.

Has anyone else had experience contributing to this particular project?

ned_batchelder2• 14 years ago

I had to make two patches because the testing style has changed slightly between the two, but also, it's guaranteed to apply better if I make them against the tip of each version.

incongruity• 14 years ago

FWIW, In my experience, the Python community has consistently been full of high-quality, very welcoming people. I don't understand why different parts of the open source world vary in such things, but they do seem to (in my limited experience) and the Python world is consistently awesome.

Your mileage may vary, of course...

(and, also fwiw, ned batchelder has a reputation for exemplifying that spirit – just trying to give credit where it's due.)

takluyver• 14 years ago

I contributed a tiny fix for a bug in Tkinter, Python's built in GUI module. There's a contributor form to sign before your first contribution can be accepted, but everyone was friendly, and it was committed quickly once I'd returned the form.

idank2• 14 years ago

This is slightly off-topic but I always wondered: why are commits in the CPython repo have their username changed from the author of the change to the person who committed it?

skrebbel• 14 years ago

Completely guessing here, but it might be a tool thing. For example it's roughly what happens when I merge your code with git and then push it to subversion with git-svn.

shuzchen2• 14 years ago

What you describe is a probable scenario, but I think the real reason is in the patch process. Contributions in python don't appear to happen pull request style ("here's my repo with my changes, please merge that in"), but rather through submitting patches ("here's a .patch file with my changes, please apply that to tip").

So what happens isn't that someone with write access merges in changes (which, unless tools get in the way, should preserve authorship), but applies the patch - which attributes that code change to the person applying the patch.

the_cat_kittles• 14 years ago

I know you from irc! good work sir!

jstanley• 14 years ago

"I discovered this working on code that grades student submissions at edX"

It sounds to me like the guy who submitted 40k spaces is the one who discovered the bug ;). Congrats on getting your patch accepted though!

slurgfest• 14 years ago

I wonder if PyPy has a similar bug - since PyPy can do a much better job at sandboxing Python programs, it might be especially important how it handles tokenization of those programs.

I fixed a bug in Python

Related Stories