MojoJolo / textteaser

TextTeaser is an automatic summarization algorithm.
MIT License
1.97k stars 251 forks source link

Set proper encoding for the source files #3

Closed netvisao closed 10 years ago

netvisao commented 10 years ago

sbt compile outputs the following on an Ubuntu 64bits

[info] Loading project definition from /home/mea/Development/sandbox/textteaser/project [info] Set current project to textteaser (in build file:/home/mea/Development/sandbox/textteaser/) [info] Compiling 9 Scala sources to /home/mea/Development/sandbox/textteaser/target/scala-2.10/classes... [error] IO error while decoding /home/mea/Development/sandbox/textteaser/src/main/scala/com/textteaser/summarizer/Main.scala with UTF-8 [error] Please try specifying another one using the -encoding option [error] one error found error Compilation failed [error] Total time: 3 s, completed Oct 12, 2013 11:46:31 AM

melpomene commented 10 years ago

I have the same issue on mac, just converted the Main.scala file to UTF-8 with intellij and problem was solved.

netvisao commented 10 years ago

I think it has to do with the files being committed from a windows system. Dos2unix did not do the full trick for me. I wonder if there is another cli solution to it.

Sent via a mobile device. On Oct 12, 2013 12:02 PM, "Christopher Käck" notifications@github.com wrote:

I have the same issue on mac, just converted the Main.scala file to UTF-8 with intellij and problem was solved.

— Reply to this email directly or view it on GitHubhttps://github.com/MojoJolo/textteaser/issues/3#issuecomment-26200112 .

tkroman commented 10 years ago

Well, the problem is with line #30 (ÒpitchÓ word). My iconv doesn't seem to be able to handle it as well.

tkroman commented 10 years ago

I fixed this in #6

MojoJolo commented 10 years ago

Thanks @cdshines. I also encountered this issue. Will check it out later and merge it.

jmazin commented 10 years ago

Hi -- I think the issue in this case is the cut-and-pasted text at Main.scala:30, specifically the use of the curly double-quotes: and .

MojoJolo commented 10 years ago

Sorry about the encoding issues. Some clumsy issues right there. Anyways. @cdshines fixed it and I merged it already. Thanks. Will close this issue.