[Adium-devl] Version Control
Colin Barrett
timber at lava.net
Thu Feb 7 10:51:47 UTC 2008
On Feb 6, 2008, at 9:21 PM, Peter Hosey wrote:
> Evan, as Colin noted, works on both Adium and Pidgin. So, if he
> commits+pushes something to Pidgin's hypothetical hg repository, he
> would use his @pidgin.im address; when he pulls it into Adium's
> repo, it will keep that address, and we'll know that he committed it
> as a Pidgin developer. When he commits+pushes directly to Adium,
> he'd use his @adiumx.com address, and we'll know that he committed
> it as an Adium developer.
How is this useful? Evan committed it. That's all we really need to
know, right? What if he commits a patch in Adium's libpurple
repository which then gets pulled into the main pidgin one? Which
email address should he use then?
> More generally, a project-identifying email address clearly says at
> a glance that the committer is or is not affiliated with our project
> or Pidgin. We don't even need to look at the username: if the domain
> is either adiumx.com or pidgin.im, it's an official patch;
> otherwise, it's a user-submitted patch.
How is this useful? Either the patch is in the main repository or it
isn't. What about this situation:
James Hacker takes an interest in the Adium project. He starts
submitting patches, initially under j.random at hacker.com. Eventually,
he becomes an official adium developer, and his @adiumx.com email is james at adiumx.com
.
He's still the same person. The purpose of having an email address in
the commit name, as you correctly argue, is to uniquely identify
someone. James is still James, wether or not he's an Adium developer
or not.
Or what about the case where someone "retires" from adium development,
but still submits a few patches now and again. What email address
should *they* use? It's not as simple a situation as you present, and
honestly it's much simpler to have the email address have no semantic
value other than "an email address people can use to contact you."
-Colin
More information about the devel
mailing list