From d2524a53bbd219ee6d069006fd243a5bcda0245b Mon Sep 17 00:00:00 2001
From: Michael Vogt <mvo@debian.org>
Date: Tue, 26 Nov 2013 09:22:40 +0100
Subject: add syntax check for sources.list

---
 cmdline/apt.cc | 19 ++++++++++++++++---
 1 file changed, 16 insertions(+), 3 deletions(-)

(limited to 'cmdline/apt.cc')

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;
 }
-- 
cgit v1.2.3