Ticket #295 (closed task: fixed)
remove ChangeLog files from mc tree
| Reported by: | slyfox | Owned by: | slavazanko |
|---|---|---|---|
| Priority: | major | Milestone: | 4.7 |
| Component: | mc-core | Version: | 4.6.2 |
| Keywords: | commited-master | Cc: | |
| Blocked By: | Blocking: | ||
| Branch state: | Votes for changeset: |
Description
GNU ChangeLog? files are result of CVS stupidpity about changeset tracking.
Currently constantly updated ChangeLogs? diverge in different branches and cause collisions(!) when 'git merge' (almost ANY SINGLE MERGE!).
Major changes can be described in NEWS file, minor changes can be autogenerated via 'git log'/'git shortlog' and friends (if needed at all).
So, $SUBJ
Change History
comment:2 Changed 16 years ago by slavazanko
- Keywords vote-slavazanko added
Well, if we will reproduce Changelog from git-log, then Changelog must be frozen. But need to keep in project as is for historical(and for respect of all contributors) reason :)
comment:5 Changed 16 years ago by kdave
- Keywords vote-kdave added
http://www.gnu.org/prep/standards/html_node/Change-Logs.html
I see one positive thing about the GNU-style changelogs: textually describing the changeset makes the author review his own changes (but this should be done anyway).
But the change description without seeing the patch is pointless (yeah, I know that somebody added a function, I can see it ...), the description belongs to VCS. Textual description of obvious things the patch does makes no sense, if there is no text about reason or explanation what it should fix. Pick your favourite pointless changelog entry yourself.
GNU ChangeLog? files are result of CVS stupidpity about changeset tracking.
Totally agree.
comment:6 Changed 16 years ago by slavazanko
- Status changed from new to accepted
- Owner set to slavazanko
comment:7 Changed 16 years ago by slavazanko
- Keywords review added; vote-angel_il vote-slavazanko vote-styx vote-kdave removed
- Type changed from enhancement to task
created branch 295_remove_changelogs (parent: master)
Initial commit: changeset:34a9677c614502482a74606af0fb3b64e49b4999
Review, please.
comment:10 Changed 16 years ago by slavazanko
- Status changed from accepted to testing
- Keywords commited-master added; review vote-styx vote-slyfox approved removed
- Resolution set to fixed
I think this is a good idea. Will have clear true haistory and get rid of this stupid collisions ...