Skip to content

usearch: Make installed binary executable#10269

Merged
adamjstewart merged 2 commits intospack:developfrom
htcf:usearch
Jan 8, 2019
Merged

usearch: Make installed binary executable#10269
adamjstewart merged 2 commits intospack:developfrom
htcf:usearch

Conversation

@erimar77
Copy link
Copy Markdown
Contributor

@erimar77 erimar77 commented Jan 7, 2019

No description provided.

@erimar77 erimar77 changed the title usearch: Make binary executable usearch: Make installed binary executable Jan 7, 2019
Copy link
Copy Markdown
Contributor

@healther healther left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why isn't this done automatically? Isn't that the job of the install method?

@erimar77
Copy link
Copy Markdown
Contributor Author

erimar77 commented Jan 8, 2019

It looks like the install method just preserves the src permissions. This is a manual download, so we can't be sure what the permissions will be. In my case, the file was saved with 644, resulting in a non-functioning package.

@adamjstewart adamjstewart merged commit 45882e1 into spack:develop Jan 8, 2019
@trecakov
Copy link
Copy Markdown
Contributor

@erimar77
Hi, I am getting this error. The downloaded file is there with the correct name.

=> Installing usearch
==> Searching for binary cache of usearch
==> Warning: No Spack mirrors are currently configured
==> No binary for usearch found: installing from source
==> Fetching file:///home/pkgmgr/usearch11.0.667_i86linux32
curl: (37) Couldn't open file /home/pkgmgr/usearch11.0.667_i86linux32
==> Fetching from file:///home/pkgmgr/usearch11.0.667_i86linux32 failed.
==> Error: FetchError: All fetchers failed for usearch-11.0.667-f7t2kwkhsnhm75nxw62razuzd77mdzkt

/software/packages/spack/lib/spack/spack/package.py:916, in do_fetch:
913 raise FetchError("Will not fetch %s" %
914 self.spec.format('$_$@'), ck_msg)
915

916 self.stage.create()
917 self.stage.fetch(mirror_only)
918 self._fetch_time = time.time() - start_time
919

Any suggestions?

@erimar77
Copy link
Copy Markdown
Contributor Author

@trecakov The only version listed is 10.0.240, can you try using that executable? You'll also need to modify the md5sum to match your file, as each download has your license information baked into it.

@erimar77 erimar77 deleted the usearch branch January 30, 2019 19:05
@trecakov
Copy link
Copy Markdown
Contributor

@erimar77 I added version 11.0.667 in the package.py.
Anyways, I tried with 10.0.240 and it worked. Thanks

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