search

adiknoth / 4deckradio

Multi-deck media playback for radio stations
MIT License
5 stars 2 forks source link

Ooops, a small fault #3

Closed karllinden closed 11 years ago

karllinden commented 11 years ago

I found a little bug in configure.ac. As the diff shows it should be withval. My bad.

adiknoth commented 11 years ago

Thanks for sending the patch. I didn't merge your branch, since your commit message was a bit broad. ;)

Like in the Linux kernel, I like concise yet precise commit messages, so the log can later be more helpful when reviewing changes, e.g. for the sake of bug hunting.

Instead of telling you to change the commit message, I simply did it myself but cited you in the commit.

Hope that's ok for you.

karllinden commented 11 years ago

OK! Splendid!