Conversation
(cherry picked from commit bc2b630)
cherry-picked from 36e32d2
Remove one of calls to transport.connecting() as this causes an assert with SslTransport (cherry picked from commit f8ca35f)
…esponseTest.test_request_response ........ fail Fix to reactor.py, check for lack of SSL domain. (cherry picked from commit e31df01)
Having switched to using git-archive, we end up creating a tar that uses the PAX-based scheme for supporting long pathnames. This causes issues for a number of extraction tools (ant, 7-zip, etc.) which only support the GNU tar format. We should stick with GNU tar, as used for previous qpid-proton releases. Also update the script to revert to using a suffix of .tar.gz (rather than .tgz) for the source distribution. Closes #14
* when receiving an attach with a handle > handle-max, the connection should be closed * when attempting to allocate a new local handle when handle-max are already in-use should throw an Exception * the default of 1024 handle-max is quite small when proton-c appears to default to 4294967295 - increase it more modestly to 65536 Closes #16 (cherry picked from commit f937ccd)
Still the encoder wasn't 100% compliant in 0.9. Also fixup the unittest to run a touch faster. (cherry picked from commit 836cf27)
The getter was incorrectly dividing the value in half before returning it. (cherry picked from commit 677ea33)
Check for no Headers object and create one as needed. (cherry picked from commit b541ad0)
(cherry picked from commit 1e4b121)
(cherry picked from commit fd26ec6)
…losed state, instead create a new object and ensure the old links also get freed. Also fixes similar behaviour as in PROTON-850. This closes #21 (cherry picked from commit f2d7d66)
(cherry picked from commit aa5ea2b)
… be run (cherry picked from commit 8235ba1)
… a tutorial to accompany the examples. (cherry picked from commit 4653cdc)
Please enter the commit message for your changes. Lines starting (cherry picked from commit df2cd6c)
(cherry picked from commit 65aa64c)
(cherry picked from commit 5bf533c)
(cherry picked from commit 450b8ba)
* remove out-of-date product information from the README and include the more current information from the website * add a new INSTALL file to contain the build and install instructions from the existing README * move all development and testing related info from that to the DEVELOPERS file * better explain how to run the various test suites and how they work across proton-c and proton-j in the developers file * other misc improvements across the contents of the files * rename DEVELOPERS and INSTALL to have .md suffixes for better rendering on GitHub * add a symlink from README --> README.md for better landing page rendering on GitHub (cherry picked from commit b532cf2)
(cherry picked from commit 1aa7bce)
(cherry picked from commit 425c008)
The system-installed Perl, Python etc. are all built as universal binaries on Mac and so we need to build libqpid-proton to similarly contain both architectures. This allows the bindings to link successfully. (cherry picked from commit a509195)
(cherry picked from commit da117ea)
(cherry picked from commit 9ab060f)
(cherry picked from commit 0c1de24)
(cherry picked from commit 2d899d1)
(cherry picked from commit 0d266c6)
As of now, it's not possible to install python-qpid-proton if libqpid-proton is not present in the system. To be more precises, it's possible to build python-qpid-proton using cmake, upload it and beg to the gods of OPs that the required (and correct) shared library will be present in the system. This patch adds to python-qpid-proton the ability to download, build and install qpid-proton if the required version is not present in the system. It does this by checking - using pkg-config - whether the required version is installed and if not, it goes to downloading the package from the official apache source and builds it using cmake. As nasty as it sounds, this process is not strange in the Python community. Very famous - and way more used - libraries like PyZMQ (from which this work took lots of inspiration) do this already in a fairly more complex way. This first step is quite simple, it checks, downloads and builds using the standard tools. It's enabled just for linux and it does not use fancy flags. Future enhancements could take care of improving the implementation and extending it to support other systems. (cherry picked from commit 2e2ba9c)
(cherry picked from commit 59afc55)
(cherry picked from commit c1f9ed4)
(cherry picked from commit 0b7718c)
Instead of using cmake to build the downloaded tarball, rely on python for building and installing the library. This process is not only cleaner but also plays nicer with the environment we're trying to give support to. In addition to the above, this plays nicer with the user and removes a requirement for having the library installed. Note that the built library will no longer overwrite system libraries but it'll be placed along to the python bindings we're building. (cherry picked from commit 4ed130e)
…lass derived from Exception
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
10 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.