|
[
Permlink
| « Hide
]
David Hergert added a comment - 09/Oct/07 04:27 PM
I agree with the second point. I would like to be able to see the types of comments that a particular developer gets. Or at the very least, for a given review, see all the comments and their severity in a single concise view.
I am going to clone this issue to separate the two points.
Lubos,
Crucible should handle Unicode files, or at least it should be able to show the differences. Can you see diffs for these files in the FishEye pages? What version of Crucible are you using, and what version control system are you using (CVS, SVN, Perforce, etc)? Can you perhaps grab a screenshot of a review were Crucible would not show the diff, and perhaps send us a copy of the Unicode file in question? The files for which I wanted to see the difference are stored by P4 as binary files. Does this explain why Crucible does not show the differences?
Yes, if Crucible thinks the file is "binary", then it will not show a diff. At the moment, FishEye/Crucible only considers files with a file type of "text" as text files.
http://www.perforce.com/perforce/doc.072/manuals/cmdref/o.ftypes.html#1040647 Can you confirm that the files in question are UNICODE or utf16? The files are UTF-16 encoded.
So, the files are known to Perforce as binary files but are actually unicode files?
That's a little more difficult to handle. We are adding support for the unicode filetype in both unicode and regular p4 servers but we accept the server's view with regard to content being binary. Does the content have the Unicode BOM?
I made one mistake above. We use UTF-8 encoded files. It is UTF-8 BOM-less encoding.
I'm going to mark this as wontfix, at least for now. If the repository thinks the file is binary, FishEye will treat the file as binary regardless of the actual content. Determining the possible content type of an arbitrary binary file is not something FishEye will do in the short term. We are considering some solutions in the longer term (custom file type handlers, for example).
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||