[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#857914: marked as done (unblock: rekall/1.6.0+dfsg-2)



Your message dated Thu, 16 Mar 2017 19:06:58 +0100
with message-id <7cc31ac0-83ec-7069-a9f9-aaa87fbce981@debian.org>
and subject line Re: Bug#857914: unblock: rekall/1.6.0+dfsg-2
has caused the Debian Bug report #857914,
regarding unblock: rekall/1.6.0+dfsg-2
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
857914: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=857914
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message ---
Package: release.debian.org
Severity: normal
User: release.debian.org@packages.debian.org
Usertags: unblock

Please unblock package rekall. I have fixed #856926 as advised by
relaxing the versioned dependencies via an additional patch.
Please see attached debdiff against 1.6.0+dfsg-1 which is currently
in testing.

For the record, there are also some small changes resulting from
the use of gbp's patch-queue feature, which was used for the first
time here. It moved around some patch contents and changed some patch
headers.

If that is not acceptable for an unblock please just let me know,
I'll make a new -3 upload reverting the patch modifications.

unblock rekall/1.6.0+dfsg-2
diff -Nru rekall-1.6.0+dfsg/debian/changelog rekall-1.6.0+dfsg/debian/changelog
--- rekall-1.6.0+dfsg/debian/changelog	2016-12-18 16:37:45.000000000 +0100
+++ rekall-1.6.0+dfsg/debian/changelog	2017-03-14 12:27:00.000000000 +0100
@@ -1,3 +1,11 @@
+rekall (1.6.0+dfsg-2) unstable; urgency=medium
+
+  * Initialize patches for use with patch-queue.
+  * Fix strict versioned dependencies in setup.py.
+    Closes: #856926
+
+ -- Sascha Steinbiss <satta@debian.org>  Tue, 14 Mar 2017 12:27:00 +0100
+
 rekall (1.6.0+dfsg-1) unstable; urgency=low
 
   [ Hilko Bengen and Sascha Steinbiss ]
diff -Nru rekall-1.6.0+dfsg/debian/patches/0001-setup.py-Remove-custom-CleanCommand.patch rekall-1.6.0+dfsg/debian/patches/0001-setup.py-Remove-custom-CleanCommand.patch
--- rekall-1.6.0+dfsg/debian/patches/0001-setup.py-Remove-custom-CleanCommand.patch	2016-12-18 16:37:45.000000000 +0100
+++ rekall-1.6.0+dfsg/debian/patches/0001-setup.py-Remove-custom-CleanCommand.patch	2017-03-14 12:26:48.000000000 +0100
@@ -3,26 +3,27 @@
 Subject: setup.py: Remove custom CleanCommand
 
 ---
- rekall-core/setup.py | 20 --------------------
- 1 file changed, 20 deletions(-)
+ rekall-agent/setup.py | 19 -------------------
+ rekall-core/setup.py  | 20 --------------------
+ rekall-gui/setup.py   | 20 --------------------
+ 3 files changed, 59 deletions(-)
 
---- a/rekall-core/setup.py
-+++ b/rekall-core/setup.py
-@@ -117,32 +117,12 @@
-         print "\n".join(sorted(result))
+diff --git a/rekall-agent/setup.py b/rekall-agent/setup.py
+index 10c292e..8a9519e 100644
+--- a/rekall-agent/setup.py
++++ b/rekall-agent/setup.py
+@@ -55,28 +55,9 @@ data_files = (find_data_files("test_data") +
+               find_data_files("messages"))
  
  
 -class CleanCommand(Command):
 -    description = ("custom clean command that forcefully removes "
 -                   "dist/build directories")
 -    user_options = []
--
 -    def initialize_options(self):
 -        self.cwd = None
--
 -    def finalize_options(self):
 -        self.cwd = os.getcwd()
--
 -    def run(self):
 -        if os.getcwd() != self.cwd:
 -            raise RuntimeError('Must be in package root: %s' % self.cwd)
@@ -30,31 +31,35 @@
 -        os.system('rm -rf ./build ./dist')
 -
 -
- commands = {}
- commands["pip_upgrade"] = PIPUpgrade
+-commands = {}
 -commands["clean"] = CleanCommand
- 
+-
  setup(
-     name="rekall-core",
+     name="rekall_agent",
      version=VERSION["pep440"],
 -    cmdclass=commands,
      description=rekall_description,
-     long_description=open(os.path.join(current_directory, "README.rst")).read(),
+     long_description="The DFIR agent component of the Rekall framework.",
      license="GPL",
---- a/rekall-agent/setup.py
-+++ b/rekall-agent/setup.py
-@@ -55,28 +55,9 @@
-               find_data_files("messages"))
+diff --git a/rekall-core/setup.py b/rekall-core/setup.py
+index e9a26cf..7261e81 100755
+--- a/rekall-core/setup.py
++++ b/rekall-core/setup.py
+@@ -117,32 +117,12 @@ class PIPUpgrade(Command):
+         print "\n".join(sorted(result))
  
  
 -class CleanCommand(Command):
 -    description = ("custom clean command that forcefully removes "
 -                   "dist/build directories")
 -    user_options = []
+-
 -    def initialize_options(self):
 -        self.cwd = None
+-
 -    def finalize_options(self):
 -        self.cwd = os.getcwd()
+-
 -    def run(self):
 -        if os.getcwd() != self.cwd:
 -            raise RuntimeError('Must be in package root: %s' % self.cwd)
@@ -62,19 +67,22 @@
 -        os.system('rm -rf ./build ./dist')
 -
 -
--commands = {}
+ commands = {}
+ commands["pip_upgrade"] = PIPUpgrade
 -commands["clean"] = CleanCommand
--
+ 
  setup(
-     name="rekall_agent",
+     name="rekall-core",
      version=VERSION["pep440"],
 -    cmdclass=commands,
      description=rekall_description,
-     long_description="The DFIR agent component of the Rekall framework.",
+     long_description=open(os.path.join(current_directory, "README.rst")).read(),
      license="GPL",
+diff --git a/rekall-gui/setup.py b/rekall-gui/setup.py
+index 8f8b46b..d4b08bf 100755
 --- a/rekall-gui/setup.py
 +++ b/rekall-gui/setup.py
-@@ -54,29 +54,9 @@
+@@ -54,29 +54,9 @@ data_files = (
      find_data_files_directory('rekall_gui/plugins/webconsole/static')
  )
  
diff -Nru rekall-1.6.0+dfsg/debian/patches/0002-fix-version.py rekall-1.6.0+dfsg/debian/patches/0002-fix-version.py
--- rekall-1.6.0+dfsg/debian/patches/0002-fix-version.py	2016-12-18 16:37:45.000000000 +0100
+++ rekall-1.6.0+dfsg/debian/patches/0002-fix-version.py	2017-03-14 12:26:48.000000000 +0100
@@ -1,8 +1,16 @@
-Description: read correct version file
-Author: Sascha Steinbiss <satta@debian.org>
+From: Debian Forensics <forensics-devel@lists.alioth.debian.org>
+Date: Tue, 14 Mar 2017 10:38:27 +0100
+Subject: fix-version.py
+
+---
+ rekall-gui/setup.py | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/rekall-gui/setup.py b/rekall-gui/setup.py
+index d4b08bf..e64063c 100755
 --- a/rekall-gui/setup.py
 +++ b/rekall-gui/setup.py
-@@ -27,7 +27,7 @@
+@@ -27,7 +27,7 @@ import os
  from setuptools import find_packages, setup, Command
  
  VERSION_ENV = {}
diff -Nru rekall-1.6.0+dfsg/debian/patches/0003-fix-module-paths.patch rekall-1.6.0+dfsg/debian/patches/0003-fix-module-paths.patch
--- rekall-1.6.0+dfsg/debian/patches/0003-fix-module-paths.patch	2016-12-18 16:37:45.000000000 +0100
+++ rekall-1.6.0+dfsg/debian/patches/0003-fix-module-paths.patch	2017-03-14 12:26:48.000000000 +0100
@@ -1,5 +1,13 @@
-Description: fix module paths
-Author: Sascha Steinbiss <satta@debian.org>
+From: Debian Forensics <forensics-devel@lists.alioth.debian.org>
+Date: Tue, 14 Mar 2017 10:38:27 +0100
+Subject: fix-module-paths
+
+---
+ rekall-core/rekall/plugins/__init__.py | 30 +++++++++++++++---------------
+ 1 file changed, 15 insertions(+), 15 deletions(-)
+
+diff --git a/rekall-core/rekall/plugins/__init__.py b/rekall-core/rekall/plugins/__init__.py
+index 340e22e..0f64b7b 100644
 --- a/rekall-core/rekall/plugins/__init__.py
 +++ b/rekall-core/rekall/plugins/__init__.py
 @@ -1,18 +1,18 @@
diff -Nru rekall-1.6.0+dfsg/debian/patches/0004-resource-paths.patch rekall-1.6.0+dfsg/debian/patches/0004-resource-paths.patch
--- rekall-1.6.0+dfsg/debian/patches/0004-resource-paths.patch	2016-12-18 16:37:45.000000000 +0100
+++ rekall-1.6.0+dfsg/debian/patches/0004-resource-paths.patch	2017-03-14 12:26:48.000000000 +0100
@@ -1,13 +1,33 @@
-Description: fix resource paths
- We like to have the data files available during the dh_auto_test phase, before
- dh_auto_install. This means they need to be copied into the build directory,
- which in setuptools is done by moving them into the package directory and
- declaring them as package_data instead of data_files. After some adjustment of
- the pkg_resources usage, this seems to work.
-Author: Sascha Steinbiss <satta@debian.org>
+From: Debian Forensics <forensics-devel@lists.alioth.debian.org>
+Date: Tue, 14 Mar 2017 10:38:27 +0100
+Subject: resource-paths
+
+---
+ rekall-agent/rekall_agent/messages/agent.py | 4 ++--
+ rekall-agent/setup.py                       | 5 +++--
+ rekall-core/rekall/resources.py             | 4 ++--
+ 3 files changed, 7 insertions(+), 6 deletions(-)
+
+diff --git a/rekall-agent/rekall_agent/messages/agent.py b/rekall-agent/rekall_agent/messages/agent.py
+index 1c9f41b..2f2c6ea 100644
+--- a/rekall-agent/rekall_agent/messages/agent.py
++++ b/rekall-agent/rekall_agent/messages/agent.py
+@@ -11,8 +11,8 @@ from rekall import resources
+ from rekall_agent import serializer
+ 
+ # Get field definitions from messages.yaml.
+-path = resources.get_resource("messages.yaml", "rekall-agent",
+-                              prefix="messages")
++path = resources.get_resource("messages.yaml", "rekall_agent",
++                              prefix="message_data")
+ 
+ DEFINITIONS = yaml.safe_load(open(path, "rb").read())
+ 
+diff --git a/rekall-agent/setup.py b/rekall-agent/setup.py
+index 8a9519e..4a00e37 100644
 --- a/rekall-agent/setup.py
 +++ b/rekall-agent/setup.py
-@@ -72,11 +72,12 @@
+@@ -72,11 +72,12 @@ setup(
      ],
      package_dir={'.': 'rekall_agent'},
      packages=find_packages('.'),
@@ -22,22 +42,11 @@
 +    #zip_safe=False,
      install_requires=install_requires,
  )
---- a/rekall-agent/rekall_agent/messages/agent.py
-+++ b/rekall-agent/rekall_agent/messages/agent.py
-@@ -11,8 +11,8 @@
- from rekall_agent import serializer
- 
- # Get field definitions from messages.yaml.
--path = resources.get_resource("messages.yaml", "rekall-agent",
--                              prefix="messages")
-+path = resources.get_resource("messages.yaml", "rekall_agent",
-+                              prefix="message_data")
- 
- DEFINITIONS = yaml.safe_load(open(path, "rb").read())
- 
+diff --git a/rekall-core/rekall/resources.py b/rekall-core/rekall/resources.py
+index ed31386..ec9e3d7 100644
 --- a/rekall-core/rekall/resources.py
 +++ b/rekall-core/rekall/resources.py
-@@ -64,10 +64,10 @@
+@@ -64,10 +64,10 @@ def get_resource(filename, package="rekall-core", prefix="resources"):
  
  def _get_pkg_resource(filename, package, prefix):
      """Query pkg_resources for the location of the filename."""
diff -Nru rekall-1.6.0+dfsg/debian/patches/0005-use-debian-yara-capstone.patch rekall-1.6.0+dfsg/debian/patches/0005-use-debian-yara-capstone.patch
--- rekall-1.6.0+dfsg/debian/patches/0005-use-debian-yara-capstone.patch	2016-12-18 16:37:45.000000000 +0100
+++ rekall-1.6.0+dfsg/debian/patches/0005-use-debian-yara-capstone.patch	2017-03-14 12:26:48.000000000 +0100
@@ -1,8 +1,16 @@
-Description: use Debian's yara and capstone packages
-Author: Sascha Steinbiss <satta@debian.org>
+From: Debian Forensics <forensics-devel@lists.alioth.debian.org>
+Date: Tue, 14 Mar 2017 10:38:27 +0100
+Subject: use-debian-yara-capstone
+
+---
+ rekall-core/setup.py | 4 ++--
+ 1 file changed, 2 insertions(+), 2 deletions(-)
+
+diff --git a/rekall-core/setup.py b/rekall-core/setup.py
+index 7261e81..6987948 100755
 --- a/rekall-core/setup.py
 +++ b/rekall-core/setup.py
-@@ -63,8 +63,8 @@
+@@ -63,8 +63,8 @@ install_requires = [
      "pyparsing==2.1.5",
      "pytz == 2016.4",
      "psutil >= 4.0, < 5.0",
diff -Nru rekall-1.6.0+dfsg/debian/patches/0006-remove-script-wih-suffix.patch rekall-1.6.0+dfsg/debian/patches/0006-remove-script-wih-suffix.patch
--- rekall-1.6.0+dfsg/debian/patches/0006-remove-script-wih-suffix.patch	2016-12-18 16:37:45.000000000 +0100
+++ rekall-1.6.0+dfsg/debian/patches/0006-remove-script-wih-suffix.patch	2017-03-14 12:26:48.000000000 +0100
@@ -1,11 +1,16 @@
-Description: remove script with suffix
- Debian policy discourages scripts with language suffixes in $PATH visible
- locations. This patch removes rekal.py from the path since /usr/bin/rekall and
- /usr/bin/rekal already exist.
-Author: Sascha Steinbiss <satta@debian.org>
+From: Debian Forensics <forensics-devel@lists.alioth.debian.org>
+Date: Tue, 14 Mar 2017 10:38:27 +0100
+Subject: remove-script-wih-suffix
+
+---
+ rekall-core/setup.py | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/rekall-core/setup.py b/rekall-core/setup.py
+index 6987948..858319b 100755
 --- a/rekall-core/setup.py
 +++ b/rekall-core/setup.py
-@@ -135,7 +135,7 @@
+@@ -135,7 +135,7 @@ setup(
          "Operating System :: OS Independent",
          "Programming Language :: Python",
      ],
diff -Nru rekall-1.6.0+dfsg/debian/patches/0007-fix-spelling.patch rekall-1.6.0+dfsg/debian/patches/0007-fix-spelling.patch
--- rekall-1.6.0+dfsg/debian/patches/0007-fix-spelling.patch	2016-12-18 16:37:45.000000000 +0100
+++ rekall-1.6.0+dfsg/debian/patches/0007-fix-spelling.patch	2017-03-14 12:26:48.000000000 +0100
@@ -1,8 +1,16 @@
-Description: fix spelling
-Author: Sascha Steinbiss <satta@debian.org>
+From: Debian Forensics <forensics-devel@lists.alioth.debian.org>
+Date: Tue, 14 Mar 2017 10:38:27 +0100
+Subject: fix-spelling
+
+---
+ rekall-core/rekall/args.py | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/rekall-core/rekall/args.py b/rekall-core/rekall/args.py
+index 19732a4..681fa93 100755
 --- a/rekall-core/rekall/args.py
 +++ b/rekall-core/rekall/args.py
-@@ -44,7 +44,7 @@
+@@ -44,7 +44,7 @@ config.DeclareOption("--plugin", default=[], type="ArrayStringParser",
  
  config.DeclareOption(
      "-h", "--help", default=False, type="Boolean",
diff -Nru rekall-1.6.0+dfsg/debian/patches/0008-remove-fixed-versions-in-setup.py.patch rekall-1.6.0+dfsg/debian/patches/0008-remove-fixed-versions-in-setup.py.patch
--- rekall-1.6.0+dfsg/debian/patches/0008-remove-fixed-versions-in-setup.py.patch	1970-01-01 01:00:00.000000000 +0100
+++ rekall-1.6.0+dfsg/debian/patches/0008-remove-fixed-versions-in-setup.py.patch	2017-03-14 12:26:48.000000000 +0100
@@ -0,0 +1,86 @@
+From: Sascha Steinbiss <sascha.steinbiss@dcso.de>
+Date: Tue, 14 Mar 2017 10:42:44 +0100
+Subject: remove fixed versions in setup.py
+
+---
+ rekall-agent/setup.py | 16 ++++++++--------
+ rekall-core/setup.py  | 36 ++++++++++++++++++------------------
+ 2 files changed, 26 insertions(+), 26 deletions(-)
+
+diff --git a/rekall-agent/setup.py b/rekall-agent/setup.py
+index 4a00e37..f653005 100644
+--- a/rekall-agent/setup.py
++++ b/rekall-agent/setup.py
+@@ -41,14 +41,14 @@ def find_data_files(source):
+     return result
+ 
+ install_requires = [
+-    "rekall-core >= 1.6.0rc1, < 1.7",
+-    "requests==2.11.1",
+-    "httplib2==0.9.2",
+-    "oauth2client==3.0.0",
+-    "cryptography==1.4",
+-    "filelock==2.0.6",
+-    "pathlib==1.0.1",
+-    "portpicker==1.1.1"
++    "rekall-core",
++    "requests >= 2.11.1",
++    "httplib2",
++    "oauth2client",
++    "cryptography",
++    "filelock",
++    "pathlib",
++    "portpicker"
+ ]
+ 
+ data_files = (find_data_files("test_data") +
+diff --git a/rekall-core/setup.py b/rekall-core/setup.py
+index 858319b..9868971 100755
+--- a/rekall-core/setup.py
++++ b/rekall-core/setup.py
+@@ -51,27 +51,27 @@ def find_data_files(source):
+ # approach ensures that any Rekall version will always work as tested - even
+ # when external packages are upgraded in an incompatible way.
+ install_requires = [
+-    "artifacts == 20160114",
+-    "pyaff4 >= 0.24, < 0.30",
+-    "PyYAML == 3.11",
+-    "acora == 2.0",
+-    "arrow == 0.7.0",
+-    "efilter == 1!1.3",
+-    "intervaltree == 2.1.0",
+-    "pycrypto == 2.6.1",
+-    "pyelftools == 0.24",
+-    "pyparsing==2.1.5",
+-    "pytz == 2016.4",
+-    "psutil >= 4.0, < 5.0",
+-    "capstone >= 3.0.4",
+-    "yara-python >= 3.4.0",
+-    "sortedcontainers == 1.4.4",
+-    "pytsk3 == 20160721",
+-    "ipaddr==2.1.11",
++    "artifacts",
++    "pyaff4",
++    "PyYAML",
++    "acora",
++    "arrow",
++    "efilter",
++    "intervaltree",
++    "pycrypto",
++    "pyelftools",
++    "pyparsing",
++    "pytz",
++    "psutil",
++    "capstone",
++    "yara-python",
++    "sortedcontainers",
++    "pytsk3",
++    "ipaddr",
+ 
+     # Version 2.5.0 is broken with pyinstaller.
+     # https://github.com/pyinstaller/pyinstaller/issues/1848
+-    "python-dateutil == 2.5.3",
++    "python-dateutil",
+ ]
+ 
+ if "VIRTUAL_ENV" not in os.environ:
diff -Nru rekall-1.6.0+dfsg/debian/patches/series rekall-1.6.0+dfsg/debian/patches/series
--- rekall-1.6.0+dfsg/debian/patches/series	2016-12-18 16:37:45.000000000 +0100
+++ rekall-1.6.0+dfsg/debian/patches/series	2017-03-14 12:26:48.000000000 +0100
@@ -5,3 +5,4 @@
 0005-use-debian-yara-capstone.patch
 0006-remove-script-wih-suffix.patch
 0007-fix-spelling.patch
+0008-remove-fixed-versions-in-setup.py.patch

--- End Message ---
--- Begin Message ---
On 16/03/17 11:40, Sascha Steinbiss wrote:
> Package: release.debian.org
> Severity: normal
> User: release.debian.org@packages.debian.org
> Usertags: unblock
> 
> Please unblock package rekall. I have fixed #856926 as advised by
> relaxing the versioned dependencies via an additional patch.
> Please see attached debdiff against 1.6.0+dfsg-1 which is currently
> in testing.
> 
> For the record, there are also some small changes resulting from
> the use of gbp's patch-queue feature, which was used for the first
> time here. It moved around some patch contents and changed some patch
> headers.
> 
> If that is not acceptable for an unblock please just let me know,
> I'll make a new -3 upload reverting the patch modifications.
> 
> unblock rekall/1.6.0+dfsg-2

Already unblocked:

Hints in place:
==> nthykier
  #2017-03-16
  # #856926
  age-days 2 rekall/1.6.0+dfsg-2
  unblock rekall/1.6.0+dfsg-2

Cheers,
Emilio

--- End Message ---

Reply to: