Project

General

Profile

Actions

Defect #11868

closed

Git and Mercurial diff displays deleted files as /dev/null

Added by Humberto Anjos over 11 years ago. Updated over 11 years ago.

Status:
Closed
Priority:
Normal
Category:
SCM
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed
Affected version:

Description

It's the same bug reported here with ChiliProject: https://www.chiliproject.org/issues/589

Basically, in the diff view of git commits, deleted files show up as changes to the file /dev/null.

I'm using the Bitnami Redmine Stack, version 2.0.3-1. Administration > Information shows:

Environment:
  Redmine version                          2.0.3.stable
  Ruby version                             1.8.7 (x86_64-linux)
  Rails version                            3.2.6
  Environment                              production
  Database adapter                         MySQL

Actions #1

Updated by Toshi MARUYAMA over 11 years ago

  • Category set to SCM
Actions #2

Updated by Jean-Philippe Lang over 11 years ago

What would be the expected behaviour?

Actions #3

Updated by Humberto Anjos over 11 years ago

Jean-Philippe Lang wrote:

What would be the expected behaviour?

To show the original file names, with some indication that the files are no longer there.

An example of reasonable behavior can be seen here: in this commit, the files Events.cs and Utilities.cs no longer exist. When one clicks in Show Diff Stats, there is an icon showing that those files were removed.

P.S. Optionally, I wouldn't even show their contents too (my diff got truncated here, since some of the removed files were quite long...), but that's an extra :)

Actions #4

Updated by Jean-Philippe Lang over 11 years ago

  • Subject changed from /dev/null in commit-diff-views to Git diff displays deleted files as /dev/null
  • Status changed from New to Closed
  • Assignee set to Jean-Philippe Lang
  • Target version set to 2.2.0
  • Resolution set to Fixed

Humberto Anjos wrote:

To show the original file names, with some indication that the files are no longer there.

An example of reasonable behavior can be seen here: in this commit, the files Events.cs and Utilities.cs no longer exist. When one clicks in Show Diff Stats, there is an icon showing that those files were removed.

Fixed in r10424. The indication that the file was deleted is displayed on the revision view.

P.S. Optionally, I wouldn't even show their contents too (my diff got truncated here, since some of the removed files were quite long...), but that's an extra :)

Not sure that this behaviour is desirable to everyone, maybe a seperate feature request.

Actions #5

Updated by Jean-Philippe Lang over 11 years ago

Oh, and don't forget to clear your cache if you upgrade (diff rendering is cached).

Actions #6

Updated by Toshi MARUYAMA over 11 years ago

  • Subject changed from Git diff displays deleted files as /dev/null to Git and Mercurial diff displays deleted files as /dev/null
Actions

Also available in: Atom PDF