anubhav94 / gstreamer-java

Automatically exported from code.google.com/p/gstreamer-java
0 stars 0 forks source link

Unable to build on case-insensitive file systems. #118

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Have a Mac (at least) with non-case sensitive file system type.
2. Setup project in Eclipse.
3. Attempt to compile sources

What is the expected output?
Successful build.

What do you see instead?
Name conflicts in .class files. Specifically:
BaseSink$Handoff.class and Identity$Handoff.class

Which is because the 'HANDOFF' interface and 'Handoff' class in BaseSink.java 
and Identity.java conflict on case-insensitive file systems.

Have you tried to verify this is a gstreamer-java specific issue, and not a
problem with the gstreamer framework itself?

Yup.

What version of the product are you using? On what operating system?

gstreamer-java 1.5, rev 583.
BaseSink.java 576
Identity.java 576

Please provide any additional information below.

Refactoring solves the problem.

Change:

private static class Handoff implements GstAPI.GstCallback

to something like:

private static class HandoffClass implements GstAPI.GstCallback

In those two files.

Original issue reported on code.google.com by nearw...@gmail.com on 9 Jan 2013 at 3:28

GoogleCodeExporter commented 9 years ago
Additional info:

Eclipse Juno
OS X 10.8.2 on HFS

Original comment by nearw...@gmail.com on 9 Jan 2013 at 3:32

GoogleCodeExporter commented 9 years ago
java is case sensitive, so don't use on a case insensitive fs! 

Original comment by lfar...@lfarkas.org on 9 Jan 2013 at 5:07

GoogleCodeExporter commented 9 years ago
I understand that Java is case sensitive. But that doesn't mean help when the 
file system itself isn't. The generated class files collide. Reformatting to a 
case-sensitive file system really pushes the meaning of "work around".

The fix is easy (5 or so lines need an identifier renamed), and allows it to 
build on more systems out of the box. I don't understand why you wouldn't want 
to fix this.

Original comment by nearw...@gmail.com on 9 Jan 2013 at 9:59

GoogleCodeExporter commented 9 years ago
Out of interest, can you reproduce this issue using javac? As far as I can see, 
those private inner classes are compiled to BaseSink$1 etc. Not sure if that's 
an Eclipse issue.

Original comment by neilcsmi...@googlemail.com on 9 Jan 2013 at 10:58

GoogleCodeExporter commented 9 years ago
No, I cannot reproduce on the command line. You're right, it seems to be an 
eclipse + case-insensitive FS issue.

Original comment by nearw...@gmail.com on 10 Jan 2013 at 6:54

GoogleCodeExporter commented 9 years ago
Issue 123 has been merged into this issue.

Original comment by neilcsmi...@googlemail.com on 22 Apr 2013 at 8:04

GoogleCodeExporter commented 9 years ago
r596

Original comment by lfar...@lfarkas.org on 22 Apr 2013 at 9:17