Skip to content

r-sf: fix build error#19386

Merged
adamjstewart merged 2 commits intospack:developfrom
open-estuary:fix_r-sf
Oct 22, 2020
Merged

r-sf: fix build error#19386
adamjstewart merged 2 commits intospack:developfrom
open-estuary:fix_r-sf

Conversation

@darmac
Copy link
Copy Markdown
Contributor

@darmac darmac commented Oct 20, 2020

fix build error:

configure: pkg-config proj exists, will use it
checking proj_api.h usability... no
checking proj_api.h presence... no
checking for proj_api.h... no
configure: error: proj_api.h not found in standard or given locations.
ERROR: configuration failed for package 'sf'
* removing '/home/spack-develop/opt/spack/linux-centos8-aarch64/gcc-8.3.1/r-sf-0.7-7-efvc3ydctgqief7maovseikhqjemiqbu/rlib/R/library/sf'

The configure file only recognize proj@:6.99.99, so we need change the version constraint.

@adamjstewart adamjstewart merged commit 646af81 into spack:develop Oct 22, 2020
@arm7star
Copy link
Copy Markdown

arm7star commented Jan 14, 2021

proj@6 doesn't work. Why not use proj@:6.99.99?

==> No binary for proj-6-fjz5rotzvmqfxyiwq7pwzrsibg7zghvl found: installing from source
==> Warning: There is no checksum on file to fetch proj@6 safely.
==> Fetch anyway? [y/N] y
==> Error: FetchError: All fetchers failed

/opt/spack/lib/spack/spack/package.py:1335, in do_fetch:
1332 self.spec.format('{name}{@Version}'), ck_msg)
1333
1334 self.stage.create()

1335 err_msg = None if not self.manual_download else self.download_instr
1336 self.stage.fetch(mirror_only, err_msg=err_msg)
1337 self._fetch_time = time.time() - start_time
1338

@adamjstewart
Copy link
Copy Markdown
Member

Must be a bug in the concretizer. Want to submit a PR to change it to @:6.999?

@darmac
Copy link
Copy Markdown
Contributor Author

darmac commented Jan 14, 2021

@adamjstewart Yes, I'll make a pr for that.

@darmac darmac deleted the fix_r-sf branch January 14, 2021 05:09
@darmac
Copy link
Copy Markdown
Contributor Author

darmac commented Jan 14, 2021

@adamjstewart Yes, I'll make a pr for that.

A new PR is here for this issue: #20898

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants