print an error if a new state file can't be created in apt-mark,

thanks Carl Chenet! (Closes: #521289)

diff --git a/cmdline/apt-mark b/cmdline/apt-mark
index 226d207..c44ce70 100755 (executable)
--- a/cmdline/apt-mark
+++ b/cmdline/apt-mark
@@ -31,8 +31,12 @@ def mark_unmark_automatic(filename, action, pkgs):
     " mark or unmark automatic flag"
     # open the statefile
     if os.path.exists(STATE_FILE):
-        tagfile = apt_pkg.ParseTagFile(open(STATE_FILE))
-        outfile = open(STATE_FILE+".tmp","w")
+        try:
+            tagfile = apt_pkg.ParseTagFile(open(STATE_FILE))
+            outfile = open(STATE_FILE+".tmp","w")
+        except IOError, msg:
+            print "%s, are you root?" % (msg)
+            sys.exit(1)
         while tagfile.Step():
             pkgname = tagfile.Section.get("Package")
             autoInst = tagfile.Section.get("Auto-Installed")

diff --git a/debian/changelog b/debian/changelog
index 4ac6eed..51f6cdf 100644 (file)
--- a/debian/changelog
+++ b/debian/changelog
@@ -60,6 +60,9 @@ apt (0.7.25) UNRELEASED; urgency=low
   * apt-pkg/acquire-item.cc:
     - allow also to skip the last patch if target is reached,
       thanks Bernhard R. Link! (Closes: #545699)
+  * cmdline/apt-mark:
+    - print an error if a new state file can't be created,
+      thanks Carl Chenet! (Closes: #521289)
 
   [ Chris Leick ]
   * doc/ various manpages: