Macaulay2 / M2

The primary source code repository for Macaulay2, a system for computing in commutative algebra, algebraic geometry and related fields.
https://macaulay2.com
346 stars 231 forks source link

Windows carriage return characters #3384

Open mahrud opened 3 months ago

mahrud commented 3 months ago

We should consider changing these files to not be in DOS mode:

$ grep -r $'\r' -l `find . | grep "\.m2$"` | grep -v build
./M2/submodules/frobby/scripts/hilbert.m2
./M2/Macaulay2/packages/EigenSolver.m2
./M2/Macaulay2/packages/GradedLieAlgebras/tut.m2
./M2/Macaulay2/packages/WeylGroups.m2
./M2/Macaulay2/packages/ToricVectorBundles.m2
./M2/Macaulay2/packages/NumericalAlgebraicGeometry/SYSTEMS/EXA/Vision/specialTrifocalSlices.m2
./M2/Macaulay2/packages/NumericalImplicitization.m2
./M2/Macaulay2/packages/Matroids/tests-Matroids.m2
./M2/Macaulay2/packages/ExteriorModules.m2
./M2/Macaulay2/packages/SpechtModule.m2
./M2/Macaulay2/packages/ConvexInterface.m2
./M2/Macaulay2/packages/MapleInterface.m2
./M2/Macaulay2/packages/EquivariantGB.m2
./M2/Macaulay2/packages/NumericalImplicitization/approxPoint.m2
./M2/Macaulay2/packages/CorrespondenceScrolls.m2
./M2/Macaulay2/packages/TSpreadIdeals.m2
./M2/Macaulay2/packages/VNumber.m2
./M2/Macaulay2/packages/Parametrization.m2
./M2/Macaulay2/packages/PHCpack/PHCpackDoc.m2
./M2/Macaulay2/packages/undistributed-packages/Polyhedra2.m2
./M2/Macaulay2/packages/Polyhedra.m2
./M2/Macaulay2/packages/Matroids.m2
./M2/Macaulay2/packages/OldPolyhedra.m2
./M2/Macaulay2/packages/ExteriorIdeals.m2
./M2/Macaulay2/packages/SRdeformations.m2
./M2/Macaulay2/packages/SpaceCurves.m2
./M2/Macaulay2/packages/OldToricVectorBundles.m2
./M2/Macaulay2/packages/Polyhedra/core/contains.m2
./M2/Macaulay2/packages/Polyhedra/core/cone/methods.m2
./M2/Macaulay2/packages/Polyhedra/extended/fan/methods.m2
./M2/Macaulay2/packages/Polyhedra/extended/standardConstructions.m2
./M2/Macaulay2/packages/Polyhedra/extended/minkowskiSum.m2
./M2/Macaulay2/packages/Polyhedra/extended/legacy.m2
./M2/Macaulay2/packages/Polyhedra/extended/directProduct.m2
./M2/Macaulay2/packages/Polyhedra/extended/incompare.m2
./M2/Macaulay2/packages/Polyhedra/extended/commonFace.m2
./M2/Macaulay2/packages/Polyhedra/extended/affineImages.m2
./M2/Macaulay2/packages/Polyhedra/loadFile.m2
./M2/Macaulay2/packages/AdjointIdeal.m2
./M2/Macaulay2/packages/supplanted-packages/Polyhedra.m2
./M2/Macaulay2/packages/PolyominoIdeals.m2
./M2/Macaulay2/tests/quarantine/newlines.m2
./bugs/LAcore/testSVD.m2
./bugs/LAcore/testSolve.m2
./bugs/mike/oneStep3.m2
./bugs/mike/1-schenck-os.m2
./bugs/mike/0-fgeiss-memleak.m2
mahrud commented 3 months ago

TIL git and GitHub support --ignore-revs-file and .git-blame-ignore-revs which allow viewing blame excluding certain commits. This means we can fix all the above files (and perhaps more, indenting, moving, etc.) without sacrificing blame-ability: https://docs.github.com/en/repositories/working-with-files/using-files/viewing-a-file#ignore-commits-in-the-blame-view