Skip to content

Commit

Permalink
Added MANIFEST.in to specify header files with sdist. This broke when…
Browse files Browse the repository at this point in the history
… moving to GitHub actions (different setuptools version?)
  • Loading branch information
rickardp committed Jul 7, 2020
1 parent 569dc1d commit b239916
Show file tree
Hide file tree
Showing 12 changed files with 23 additions and 11 deletions.
4 changes: 0 additions & 4 deletions .github/workflows/python-package.yml
Original file line number Diff line number Diff line change
Expand Up @@ -38,10 +38,6 @@ jobs:
if: startsWith(github.ref, 'refs/tags')
steps:
- uses: actions/checkout@v2
- name: Set up Python for dist
uses: actions/setup-python@v2
with:
python-version: 3.8
- name: Build dist
run: |
python setup.py sdist
Expand Down
5 changes: 5 additions & 0 deletions .vscode/settings.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
{
"files.associations": {
"splitstream_private.h": "c"
}
}
1 change: 1 addition & 0 deletions MANIFEST.in
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
include include/*.h
Binary file removed apa.gz
Binary file not shown.
File renamed without changes.
File renamed without changes.
13 changes: 11 additions & 2 deletions setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,17 @@
author_email="[email protected]",
version="1.2.3",
description="Splitting of (XML, JSON) objects from a continuous stream",
ext_modules=[Extension('splitstream', ['src/python/splitstream_py.c', 'src/splitstream.c', 'src/splitstream_xml.c', 'src/splitstream_json.c', 'src/splitstream_ubjson.c', 'src/mempool.c'])],
headers=['src/splitstream.h', 'src/splitstream_private.h'],
ext_modules=[Extension('splitstream',
[
'src/python/splitstream_py.c',
'src/splitstream.c',
'src/splitstream_xml.c',
'src/splitstream_json.c',
'src/splitstream_ubjson.c',
'src/mempool.c'
],
include_dirs=["include/"])],
headers=['include/splitstream.h', 'include/splitstream_private.h'],
install_requires=requirements + test_requirements,
zip_safe=False,
test_suite='test',
Expand Down
3 changes: 2 additions & 1 deletion src/python/splitstream_py.c
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,8 @@

#include <Python.h>
#include <bytesobject.h>
#include "../splitstream.h"
#include <splitstream.h>

const static int SPLITSTREAM_STATE_FLAG_DID_RETURN_DOCUMENT = 8;
const static int SPLITSTREAM_STATE_FLAG_FILE_EOF = 16;

Expand Down
2 changes: 1 addition & 1 deletion src/splitstream.c
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
* limitations under the License.
*/

#include "splitstream_private.h"
#include <splitstream_private.h>
#include <string.h>

static void AppendDoc(SplitstreamState* state, SplitstreamDocument* dest, const void* ptr, size_t length);
Expand Down
2 changes: 1 addition & 1 deletion src/splitstream_json.c
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
* limitations under the License.
*/

#include "splitstream_private.h"
#include <splitstream_private.h>

size_t SplitstreamJSONScanner(SplitstreamState* s, const char* buf, size_t len, size_t* start) {
int escapeCounter = s->counter[0];
Expand Down
2 changes: 1 addition & 1 deletion src/splitstream_ubjson.c
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
* limitations under the License.
*/

#include "splitstream_private.h"
#include <splitstream_private.h>

size_t SplitstreamUBJSONScanner(SplitstreamState* s, const char* buf, size_t len, size_t* start) {
int remainingCounter = s->counter[0], value = s->counter[1];
Expand Down
2 changes: 1 addition & 1 deletion src/splitstream_xml.c
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
* limitations under the License.
*/

#include "splitstream_private.h"
#include <splitstream_private.h>

const int COUNTER_DASH = 0;
const int COUNTER_CLOSING_BRACKET = 1;
Expand Down

0 comments on commit b239916

Please sign in to comment.