Ticket #295 (closed task: fixed)

Opened 10 years ago

Last modified 9 years ago

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:1 Changed 10 years ago by styx

I think this is a good idea. Will have clear true haistory and get rid of this stupid collisions ...

comment:2 Changed 10 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:3 Changed 10 years ago by styx

  • Keywords vote-styx added

comment:4 Changed 10 years ago by angel_il

  • Keywords vote-angel_il added

comment:5 Changed 10 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 10 years ago by slavazanko

  • Status changed from new to accepted
  • Owner set to slavazanko

comment:7 Changed 10 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:8 Changed 10 years ago by styx

  • Keywords vote-styx added

comment:9 Changed 10 years ago by slyfox

  • Keywords vote-slyfox approved added

comment:10 Changed 10 years ago by slavazanko

  • Keywords commited-master added; review vote-styx vote-slyfox approved removed
  • Status changed from accepted to testing
  • Resolution set to fixed

comment:11 Changed 10 years ago by slavazanko

  • Status changed from testing to closed
Note: See TracTickets for help on using tickets.