<div dir="ltr">Yes. I also prefer the latter, it's more diff friendly and has more consistent indentation. We should pick and indentation amount (2 or 4) and stick with it. I vote for 2, just because that's what I use. :)<div>

<br></div><div>If we could encode a style guide in a formatter, it would be much easier for us to do automatic rewriting of Cabal files, which is needed for features like `cabal freeze`</div><div><br></div><div>P.S. I think Duncan wants to do this as well, but he got stuck on some Parsec dependency like issue IIRC.</div>

<div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Feb 26, 2014 at 2:48 PM, Carter Schonwald <span dir="ltr"><<a href="mailto:carter.schonwald@gmail.com" target="_blank">carter.schonwald@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I don't see why not.  <div class="HOEnZb"><div class="h5"><span></span><br><br>On Wednesday, February 26, 2014, Daniel Trstenjak <<a href="mailto:daniel.trstenjak@gmail.com" target="_blank">daniel.trstenjak@gmail.com</a>> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Hi all,<br>
<br>
the current cabal file pretty printer creates output in the form:<br>
<br>
   ...<br>
   extra-source-files: README.md tests/inputFiles/original.cabal<br>
                       tests/inputFiles/setup-config tests/goldenFiles/*.cabal<br>
                       tests/outputFiles/.gitignore<br>
<br>
   library<br>
       build-depends: base >=3 && <5, cmdargs >=0.10.5 && <0.11,<br>
                      lens >=4.0.1 && <4.1, strict >=0.3.2 && <0.4, Cabal >=1.18.0 && <1.19<br>
       exposed-modules: CabalBounds.Args CabalBounds.Main<br>
       ...<br>
       other-modules: Paths_cabal_bounds CabalBounds.Bound<br>
                      CabalBounds.Targets CabalBounds.Dependencies CabalBounds.Drop<br>
                      CabalBounds.Update CabalBounds.Lenses<br>
       ...<br>
<br>
<br>
Would a patch be accepted that creates output that is more diff friendly?<br>
<br>
Something like:<br>
<br>
   extra-source-files: README.md<br>
                       tests/inputFiles/original.cabal<br>
                       tests/inputFiles/setup-config<br>
<br>
   library<br>
       build-depends: base >=3 && <5,<br>
                      cmdargs >=0.10.5 && <0.11,<br>
                      lens >=4.0.1 && <4.1,<br>
       exposed-modules: CabalBounds.Args<br>
                        CabalBounds.Main<br>
       ...<br>
       other-modules: Paths_cabal_bounds<br>
                      CabalBounds.Bound<br>
                      CabalBounds.Targets<br>
       ...<br>
<br>
<br>
Or perhaps:<br>
<br>
   extra-source-files:<br>
       README.md<br>
       tests/inputFiles/original.cabal<br>
       tests/inputFiles/setup-config<br>
<br>
   library<br>
       build-depends:<br>
           base >=3 && <5,<br>
           cmdargs >=0.10.5 && <0.11,<br>
           lens >=4.0.1 && <4.1,<br>
       exposed-modules:<br>
           CabalBounds.Args<br>
           CabalBounds.Main<br>
       ...<br>
       other-modules:<br>
           Paths_cabal_bounds<br>
           CabalBounds.Bound<br>
           CabalBounds.Targets<br>
       ...<br>
<br>
<br>
I think that I might prefer the second one.<br>
<br>
<br>
Greetings,<br>
Daniel<br>
_______________________________________________<br>
cabal-devel mailing list<br>
<a>cabal-devel@haskell.org</a><br>
<a href="http://www.haskell.org/mailman/listinfo/cabal-devel" target="_blank">http://www.haskell.org/mailman/listinfo/cabal-devel</a><br>
</blockquote>
</div></div><br>_______________________________________________<br>
cabal-devel mailing list<br>
<a href="mailto:cabal-devel@haskell.org">cabal-devel@haskell.org</a><br>
<a href="http://www.haskell.org/mailman/listinfo/cabal-devel" target="_blank">http://www.haskell.org/mailman/listinfo/cabal-devel</a><br>
<br></blockquote></div><br></div></div>