conda-forge / python-eccodes-feedstock

A conda-smithy repository for python-eccodes.
BSD 3-Clause "New" or "Revised" License
6 stars 16 forks source link

Fix py3k eccodes read functions #2

Closed kmuehlbauer closed 8 years ago

kmuehlbauer commented 8 years ago

While working on the ecmwf_grib patch I encountered a probelm while reading data with python-eccodes.

The problem was that also the read functions need fflush() applied, for correct work. This fixes it. Also bumbing build number.

conda-forge-linter commented 8 years ago

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

kmuehlbauer commented 8 years ago

@ocefpaf One big issue, didn't know that I also have to flush reading buffers.

Anyway, this is real hell (c-library, swig and python3) :fire:.

ocefpaf commented 8 years ago

Anyway, this is real hell (c-library, swig and python3) :fire:.

You are a brave man for doing this. I just gave up and use Python 2.7 :stuck_out_tongue_winking_eye:

Thanks for doing this! (When the patch is robust enough let's send it upstream.)

kmuehlbauer commented 8 years ago

@ocefpaf OK, this worked. I looking forward to intensively test the builds tomorrow, so merging is very much liked :+1: