Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update package to 2.18.20 #38

Closed
wants to merge 3 commits into from

Conversation

SrNetoChan
Copy link
Contributor

@SrNetoChan SrNetoChan commented Jun 7, 2018

Checklist

  • Used a fork of the feedstock to propose changes
  • Bumped the build number (if the version is unchanged)
  • Reset the build number to 0 (if the version changed)
  • Re-rendered with the latest conda-smithy
  • Ensured the license file is being packaged.

@conda-forge-linter
Copy link

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.

@SrNetoChan
Copy link
Contributor Author

@ocefpaf @ceholden This branch started to fail on circleci after the merge of my last PR (although the PR was not failing the tests)

Also, it is building fine locally.

Any idea what may be the problem?

@ocefpaf
Copy link
Member

ocefpaf commented Jun 27, 2018

It seems to be breaking here:

[ 59%] Linking CXX shared library ../../output/lib/libqgis_gui.so
[ 59%] Built target qgis_gui
make: *** [all] Error 2

I'll take a look later today.

PS: when you say locally do you mean with conda-forge's docker image or using your system libs? The former should reproduce the same breakage here but the latter may be using something installed in your computer.

@SrNetoChan
Copy link
Contributor Author

@ocefpaf wrote

PS: when you say locally do you mean with conda-forge's docker image or using your system libs? The former should reproduce the same breakage here but the latter may be using something installed in your computer.

The latter. So probably it pick up some dependency from elsewhere.

@ocefpaf
Copy link
Member

ocefpaf commented Jun 27, 2018

The latter. So probably it pick up some dependency from elsewhere.

Take a look at

https://conda-forge.org/docs/testing.html#docker-tests

for the docker image instructions. You can also mount it with

docker run --net=host -it --rm -v $(pwd):/home/ condaforge/linux-anvil /bin/sh

so you can make multiple tries in one go and debug things.

@ceholden
Copy link
Contributor

QGIS2 looks to have stopped with release 2.18.28 on Feb 14, 2019 -- is there any desire to update to it?

There is a PR to update the qt4 recipe to use conda-build 3 so it might be eventually possible to put out a new QGIS2 if there's interest. Python 2 is unsupported soon enough though and personally I've resigned myself to just porting/rewriting all my QGIS2 relevant stuff.

@SrNetoChan
Copy link
Contributor Author

I don't think we should keep maintaining a 2.18 version. And no point in wasting time with it. Let's focus on QGIS 3, 3.4 LTR and stable (3.6) if possible.

@SrNetoChan SrNetoChan closed this Mar 28, 2019
@SrNetoChan SrNetoChan deleted the update_to_2.18.20 branch June 23, 2020 22:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants