Matthew Burgess wrote:

[EMAIL PROTECTED] wrote:

Author: jim
Date: 2005-06-13 00:15:28 -0600 (Mon, 13 Jun 2005)
New Revision: 5899


In response to the other messages regarding the log message (which I see is sorted now), I also note that this commit goes against the generally accepted (though I suspect not yet formally documented) policy of committing one set of related changes at a time. That is, this commit should really have been split into 5 separate commits:

1) Fix validation errors (these shouldn't be occurring anyway - someone's not doing a `make validate' prior to commit, or it's broken)
2) Remove duplicate entries from *64-chapter.xml
3) Update to binutils-2.16.1
4) Add BUILD_CC to sparc64 glibc
5) Add LFS_TARGET32 to sparc64 build

If this guideline isn't followed, it makes it harder to review changes, and also harder to simply revert individual changes. Reverting would either involve backing out all 5 changes and re-applying 4 of them, or manually undoing the changes.

Regards,

Matt.

That's going to be a problem with the svk tools then. Because when it merges it lumps them all together. I will see if I can get it to do one commit at a time for each entry.

When I'm working on the mirror. I go through a fix a series of issues per section. I will do my best to get to work.


--
------
[EMAIL PROTECTED]
[EMAIL PROTECTED]

LFS User # 2577
Registered Linux User # 299986

--
http://linuxfromscratch.org/mailman/listinfo/lfs-book
FAQ: http://www.linuxfromscratch.org/lfs/faq.html
Unsubscribe: See the above information page

Reply via email to