add syntax check for sources.list

1 files changed, 16 insertions, 3 deletions

diff --git a/cmdline/apt.cc b/cmdline/apt.cc
index ef31d0029..38610e731 100644
--- a/cmdline/apt.cc
+++ b/cmdline/apt.cc
@@ -67,9 +67,22 @@ bool EditSources(CommandLine &CmdL)
 
    // FIXME: take hash before, 
    //        when changed display message to apt update
-   //        do syntax check after save (like visudo)
-
-   EditFileInSensibleEditor(sourceslist);
+   bool res;
+   pkgSourceList sl;
+
+   do {
+      EditFileInSensibleEditor(sourceslist);
+      _error->PushToStack();
+      res = sl.Read(sourceslist);
+      if (!res) {
+         std::string outs;
+         strprintf(outs, _("Failed to parse %s. Edit again? "),
+                   sourceslist.c_str());
+         std::cout << outs;
+         res = !YnPrompt(true);
+      }
+      _error->RevertToStack();
+   } while (res == false);
 
    return true;
 }