[allura:tickets] Ticket 7538 discussion

[Dave Brondsema <da...@brondsema.net>]

- **status**: review --> closed
- **Reviewer**: Dave Brondsema



---

** [tickets:#7538] If diff is empty, it shouldn't show "empty file" [ss7532]**

**Status:** closed
**Milestone:** unreleased
**Labels:** p3 support 42cc bitesize 
**Created:** Sat Jul 05, 2014 03:33 PM UTC by Chris Tsai
**Last Updated:** Mon Aug 31, 2015 01:57 PM UTC
**Owner:** Igor Bondarenko


[forge:site-support:#7352]

>In this revision
https://sourceforge.net/p/kaldi/code/3889/
there were some property changes on files, but this is displayed by your site in a misleading way. It says
changed /trunk/egs/wsj/s5/steps/nnet2/dump_bottleneck_features.sh
changed /trunk/egs/wsj/s5/steps/nnet2/train_tanh_bottleneck.sh
and then later on, it says

>/trunk/egs/wsj/s5/steps/nnet2/dump_bottleneck_features.sh Diff Switch to side-by-side view
Empty file
/trunk/egs/wsj/s5/steps/nnet2/train_tanh_bottleneck.sh Diff Switch to side-by-side view
Empty file

>This looks like the user deleted the file, but what really happened is that they changed the svn:executable property to have a value ('').

Seems to me that the problem is the text used when the diff returns empty. Perhaps changing to "File contents unchanged" would work.


---

Sent from forge-allura.apache.org because dev@allura.apache.org is subscribed to https://forge-allura.apache.org/p/allura/tickets/

To unsubscribe from further messages, a project admin can change settings at https://forge-allura.apache.org/p/allura/admin/tickets/options.  Or, if this is a mailing list, you can unsubscribe from the mailing list.