CLI options: debug info & sourcemap about sassc HOT 8 CLOSED

FYI I found the implementation of the line-number and the debug-info options here: sass/libsass@4d44a4d with also some bad namings in the code.

from sassc.

What i'm doing wrong with sourcemaps?
I'm using -g options and result returned equals to result that returned from compilation without options? And not creating sourcemaps file in directory
eg.:
sassc -g styles.scss > styles.css equals to sassc styles.scss > styles.css

Thank you in advice!

from sassc.

@cbmd Source map generation currently only works if you specify an output file with the -o flag. sassc -g -o styles.css styles.scss should generate a source map. If you do not specifiy an output path, setting the sourceMappingURL directive does not make much sense, as it is relative to the output path.

@emagnier As the -g options was already documented as source map flag, I've coupled the source map implementation to it. But I agree we should review the API and keep it as close as possible to the Ruby version.

from sassc.

@svnieuw: Thanks a lot for your PR, it's much clearer now :-)

Also I played a bit with the latest version and found some minor issues (tested only on Windows):

• The -l doesn't seem to work: I get option '-l' is ambiguous. However --line-numbers and --line-comments are working fine.
• Long arguments are also working with a simple dash, like -help. I don't think they should.

from sassc.

I could replace the getopt_long_only() call with getopt_long(). This will reject single dash long options, but -help would still print the help message. It even did before my change because only the first character is parsed, the help message is printed and the program exits.

The -l flag seems to work fine for me on OS X and Linux. Which compiler did you use to compile sassc on Windows?

from sassc.

I used MsysGit to compile on Windows, but I will redo some tests probably tonight.

from sassc.

I was able to reproduce the issue in a Windows VM with MsysGit. I will try to fix it this weekend.

from sassc.

Thanks for the latest PR, everything looks good now!

from sassc.