<feed xmlns='http://www.w3.org/2005/Atom'>
<title>gpgme, branch dkg/fix-T4271</title>
<subtitle>GnuPG Made Easy.</subtitle>
<id>http://git.bktus.com/gpgfrontend/gpgme/atom?h=dkg%2Ffix-T4271</id>
<link rel='self' href='http://git.bktus.com/gpgfrontend/gpgme/atom?h=dkg%2Ffix-T4271'/>
<link rel='alternate' type='text/html' href='http://git.bktus.com/gpgfrontend/gpgme/'/>
<updated>2018-12-04T23:41:16Z</updated>
<entry>
<title>python: overhaul logic of Context.decrypt()</title>
<updated>2018-12-04T23:41:16Z</updated>
<author>
<name>Daniel Kahn Gillmor</name>
<email>dkg@fifthhorseman.net</email>
</author>
<published>2018-12-04T17:44:35Z</published>
<link rel='alternate' type='text/html' href='http://git.bktus.com/gpgfrontend/gpgme/commit/?id=ac8d7238dbf165950c9844e5cb41da8eb4d37bc0'/>
<id>urn:sha1:ac8d7238dbf165950c9844e5cb41da8eb4d37bc0</id>
<content type='text'>
* lang/python/src/core.py (Context.decrypt): simplify and clarify the
logic behind handling verify=False.
* lang/python/tests/t-decrypt.py: ensure that we test verify=False

--

The function-internal variables were pretty unclear to the reader, and
the logic caused pretty nasty breakage when verify=False.

GnuPG-Bug-Id: 4271
Signed-off-by: Daniel Kahn Gillmor &lt;dkg@fifthhorseman.net&gt;
</content>
</entry>
<entry>
<title>python: ctx.decrypt() has problematic error handling</title>
<updated>2018-12-04T23:40:25Z</updated>
<author>
<name>Daniel Kahn Gillmor</name>
<email>dkg@fifthhorseman.net</email>
</author>
<published>2018-12-04T17:29:40Z</published>
<link rel='alternate' type='text/html' href='http://git.bktus.com/gpgfrontend/gpgme/commit/?id=5e21e61cfef809a7194ff94fb3aac525169a26b5'/>
<id>urn:sha1:5e21e61cfef809a7194ff94fb3aac525169a26b5</id>
<content type='text'>
* lang/python/src/core.py (Context.decrypt): document odd
error-handling behavior as a potential problem to be addressed.

Signed-off-by: Daniel Kahn Gillmor &lt;dkg@fifthhorseman.net&gt;
</content>
</entry>
<entry>
<title>python: Clarify the meaning of ctx.decrypt(verify=[])</title>
<updated>2018-12-04T23:40:24Z</updated>
<author>
<name>Daniel Kahn Gillmor</name>
<email>dkg@fifthhorseman.net</email>
</author>
<published>2018-12-04T17:27:15Z</published>
<link rel='alternate' type='text/html' href='http://git.bktus.com/gpgfrontend/gpgme/commit/?id=fefa46173e506b0e4156e73a250d6915fe229223'/>
<id>urn:sha1:fefa46173e506b0e4156e73a250d6915fe229223</id>
<content type='text'>
* lang/python/src/core.py (Context.decrypt): docstring clarification
of what it means to pass an empty list to the verify argument.

Signed-off-by: Daniel Kahn Gillmor &lt;dkg@fifthhorseman.net&gt;
</content>
</entry>
<entry>
<title>python: gpg.Context.decrypt verify_sigs and sink_result are bools</title>
<updated>2018-12-04T23:40:24Z</updated>
<author>
<name>Daniel Kahn Gillmor</name>
<email>dkg@fifthhorseman.net</email>
</author>
<published>2018-11-28T06:51:24Z</published>
<link rel='alternate' type='text/html' href='http://git.bktus.com/gpgfrontend/gpgme/commit/?id=30ddb2cabcd6e24eaa7f73d1ec099ddff5e41646'/>
<id>urn:sha1:30ddb2cabcd6e24eaa7f73d1ec099ddff5e41646</id>
<content type='text'>
Both of these function-internal variables are never used for anything
other than a binary state.  Implement them as the booleans they are.
Otherwise, casual readers of the code might think that they're
supposed to represent something other than a flag (e.g. "verify_sigs"
could mean "the signatures to verify", and "sink_result" could mean
"the place where we sink the result").

Signed-Off-By: Daniel Kahn Gillmor &lt;dkg@fifthhorseman.net&gt;
</content>
</entry>
<entry>
<title>python: clarify documentation for verify argument for Context.decrypt()</title>
<updated>2018-12-04T23:40:24Z</updated>
<author>
<name>Daniel Kahn Gillmor</name>
<email>dkg@fifthhorseman.net</email>
</author>
<published>2018-11-28T06:47:20Z</published>
<link rel='alternate' type='text/html' href='http://git.bktus.com/gpgfrontend/gpgme/commit/?id=9a1903cc42924851ce2452d7be2e5132f2f26332'/>
<id>urn:sha1:9a1903cc42924851ce2452d7be2e5132f2f26332</id>
<content type='text'>
It's easy to miss that verify can take a list of keys.  Make it more
obvious to the average python dev who reads docstrings.

Signed-off-by: Daniel Kahn Gillmor &lt;dkg@fifthhorseman.net&gt;
</content>
</entry>
<entry>
<title>python: simplify Context.decrypt()</title>
<updated>2018-12-04T23:40:24Z</updated>
<author>
<name>Daniel Kahn Gillmor</name>
<email>dkg@fifthhorseman.net</email>
</author>
<published>2018-11-28T06:22:13Z</published>
<link rel='alternate' type='text/html' href='http://git.bktus.com/gpgfrontend/gpgme/commit/?id=827a2f3ad562a5927ceb89a842779b4b8c57c367'/>
<id>urn:sha1:827a2f3ad562a5927ceb89a842779b4b8c57c367</id>
<content type='text'>
In the course of trying to address https://dev.gnupg.org/T4271, i
discovered that gpg.Context.decrypt() has a bit of superfluous code.
This changeset is intended to simplify the code without making any
functional changes.

Signed-off-by: Daniel Kahn Gillmor &lt;dkg@fifthhorseman.net&gt;
</content>
</entry>
<entry>
<title>python: sphinx modification</title>
<updated>2018-12-04T22:31:10Z</updated>
<author>
<name>Ben McGinnes</name>
<email>ben@adversary.org</email>
</author>
<published>2018-12-04T22:31:10Z</published>
<link rel='alternate' type='text/html' href='http://git.bktus.com/gpgfrontend/gpgme/commit/?id=c8a04af676d981b42172c9bdad82eb83f91c8ce6'/>
<id>urn:sha1:c8a04af676d981b42172c9bdad82eb83f91c8ce6</id>
<content type='text'>
* The make.bat file incorrectly triggers the trailing whitespace check
  in the git repo (it doesn't actually have any trailing whitespace,
  but triggers the error on every line).
* Will need to add a script to handle future org-mode conversions of
  the index page anyway, so will get that script to deal with this
  problem too by generating the make.bat file if the platform is
  detected as Windows.

Signed-off-by: Ben McGinnes &lt;ben@adversary.org&gt;
</content>
</entry>
<entry>
<title>python: Sphinx support</title>
<updated>2018-12-04T22:17:31Z</updated>
<author>
<name>Ben McGinnes</name>
<email>ben@adversary.org</email>
</author>
<published>2018-12-04T22:17:31Z</published>
<link rel='alternate' type='text/html' href='http://git.bktus.com/gpgfrontend/gpgme/commit/?id=6d8823b81162c18406d40ec59bd252cfdf1df648'/>
<id>urn:sha1:6d8823b81162c18406d40ec59bd252cfdf1df648</id>
<content type='text'>
* Added framework from sphinx-quickstart to lang/python/doc/rst/ so
  that Python developers already using Sphinx can use the generated
  reST files with existing documentation systems.
* Note that when generating source files from Org-mode, the index page
  will require manual intervention to match the Sphinx requirements.

Signed-off-by: Ben McGinnes &lt;ben@adversary.org&gt;
</content>
</entry>
<entry>
<title>Merge branch 'master' of ssh+git://playfair.gnupg.org/git/gpgme</title>
<updated>2018-12-04T21:49:41Z</updated>
<author>
<name>Ben McGinnes</name>
<email>ben@adversary.org</email>
</author>
<published>2018-12-04T21:49:41Z</published>
<link rel='alternate' type='text/html' href='http://git.bktus.com/gpgfrontend/gpgme/commit/?id=8d2621fcad3ded2a4184c00a04ca1002b4de98cc'/>
<id>urn:sha1:8d2621fcad3ded2a4184c00a04ca1002b4de98cc</id>
<content type='text'>
</content>
</entry>
<entry>
<title>python: docs update</title>
<updated>2018-12-04T21:42:08Z</updated>
<author>
<name>Ben McGinnes</name>
<email>ben@adversary.org</email>
</author>
<published>2018-12-04T21:42:08Z</published>
<link rel='alternate' type='text/html' href='http://git.bktus.com/gpgfrontend/gpgme/commit/?id=8613727f1ee985c3cfa2c815523312914f033ffd'/>
<id>urn:sha1:8613727f1ee985c3cfa2c815523312914f033ffd</id>
<content type='text'>
* Expanded the section on issues with Windows installations, with
  greater detail of which versions of Visual Studio are needed
  depending on which version of CPython is to be used.
* Included a recommendation which is a bit harsh without being totally
  prickish.
* Updated all files to not link to author's key or related data in
  order to make them all consistent with the changes in commit
  649b19688132dd315e361c0f5b63ba6d8f45996d.

Signed-off-by: Ben McGinnes &lt;ben@adversary.org&gt;
</content>
</entry>
</feed>
