Skip to content

[htslib, htscodecs] New ports#39261

Merged
JavierMatosD merged 15 commits intomicrosoft:masterfrom
teo-tsirpanis:htslib
Jul 20, 2024
Merged

[htslib, htscodecs] New ports#39261
JavierMatosD merged 15 commits intomicrosoft:masterfrom
teo-tsirpanis:htslib

Conversation

@teo-tsirpanis
Copy link
Contributor

  • Changes comply with the maintainer guide.
  • The name of the port matches an existing name for this component on https://repology.org/ if possible, and/or is strongly associated with that component on search engines.
  • Optional dependencies are resolved in exactly one way. For example, if the component is built with CMake, all find_package calls are REQUIRED, are satisfied by vcpkg.json's declared dependencies, or disabled with CMAKE_DISABLE_FIND_PACKAGE_Xxx.
  • The versioning scheme in vcpkg.json matches what upstream says.
  • The license declaration in vcpkg.json matches what upstream says.
  • The installed as the "copyright" file matches what upstream says.
  • The source code of the component installed comes from an authoritative source.
  • The generated "usage text" is accurate. See adding-usage for context.
  • The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
  • Only one version is in the new port's versions file.
  • Only one version is added to each modified port's versions file.

@jimwang118 jimwang118 added the category:new-port The issue is requesting a new library to be added; consider making a PR! label Jun 13, 2024
Copy link
Contributor

@dg0yt dg0yt left a comment

Choose a reason for hiding this comment

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

The has_${feature} variables are unused AFAICS.

@jimwang118
Copy link
Contributor

@teo-tsirpanis Thanks for your PR, Is work still being done for this PR?

@teo-tsirpanis
Copy link
Contributor Author

I will look at the Android failures later.

@teo-tsirpanis teo-tsirpanis force-pushed the htslib branch 2 times, most recently from 67dbde1 to 5537d31 Compare July 15, 2024 16:44
@teo-tsirpanis
Copy link
Contributor Author

I can't download the artifacts to check the Android failures.

@dg0yt
Copy link
Contributor

dg0yt commented Jul 15, 2024

I can't download the artifacts to check the Android failures.

Can be downloaded as always.

ld.lld: error: unable to find library -lpthread
clang-17: error: linker command failed with exit code 1 (use -v to see invocation)
make: *** [Makefile:361: libhts.so] Error 1
make: *** Waiting for unfinished jobs....

This should probably be -pthread, for most modern systems.

@teo-tsirpanis
Copy link
Contributor Author

Thanks @dg0yt! I finally found where the artifacts were located in AzDo.

@teo-tsirpanis
Copy link
Contributor Author

CI is green.

@teo-tsirpanis teo-tsirpanis marked this pull request as ready for review July 16, 2024 11:27
@jimwang118
Copy link
Contributor

All features passed with following triplet:

x64-linux

@jimwang118 jimwang118 added the info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. label Jul 17, 2024
Copy link
Contributor

@dg0yt dg0yt left a comment

Choose a reason for hiding this comment

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

(Opinion)

@teo-tsirpanis
Copy link
Contributor Author

Feedback addressed.

@JavierMatosD
Copy link
Contributor

JavierMatosD commented Jul 18, 2024

I was unable to install these ports. I'm using the a code spaces instance

% vcpkg install htslib htscodecs

CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:127 (message):
    Command failed: /usr/bin/bash -c "V=1 ./../src/1.20-30e933fbb9.clean/configure  \"--with-external-htscodecs\" \"--disable-libcurl\" \"--disable-gcs\" \"--disable-s3\" \"--disable-plugins\" \"--disable-bz2\" \"--disable-lzma\" \"--without-libdeflate\" \"--disable-silent-rules\" \"--verbose\" \"--disable-shared\" \"--enable-static\" \"--prefix=/workspaces/vcpkg/installed/x64-linux/debug\" \"--bindir=\\${prefix}/../tools/htslib/debug/bin\" \"--sbindir=\\${prefix}/../tools/htslib/debug/sbin\" \"--libdir=\\${prefix}/lib\" \"--includedir=\\${prefix}/../include\" \"--datarootdir=\\${prefix}/share/htslib\""
    Working Directory: /workspaces/vcpkg/buildtrees/htslib/x64-linux-dbg
    Error code: 1
    See logs for more information:
      /workspaces/vcpkg/buildtrees/htslib/config-x64-linux-dbg-config.log
      /workspaces/vcpkg/buildtrees/htslib/config-x64-linux-dbg-out.log
      /workspaces/vcpkg/buildtrees/htslib/config-x64-linux-dbg-err.log

Call Stack (most recent call first):
  scripts/cmake/vcpkg_configure_make.cmake:866 (vcpkg_execute_required_process)
  ports/htslib/portfile.cmake:32 (vcpkg_configure_make)
  scripts/ports.cmake:192 (include)

/workspaces/vcpkg/buildtrees/htslib/config-x64-linux-dbg-err.log

configure: WARNING: unrecognized options: --disable-silent-rules, --disable-shared, --enable-static
configure: error: libhtscodecs development files not found: missing library

You asked to use an external htscodecs library, but do not have the
required header / library files.  You either need to supply these and
if necessary set CPPFLAGS and LDFLAGS so the compiler can find them;
or configure using --without-external-htscodecs to build the required
functions from the htscodecs submodule.

It looks like it can't find:

  • htscodecs/htscodecs.h
  • libhtscodecs.so

@JavierMatosD JavierMatosD marked this pull request as draft July 18, 2024 19:07
@teo-tsirpanis
Copy link
Contributor Author

Weird. Do these files exist? Did htscodecs build correctly? Can you try installing only htslib?

@teo-tsirpanis
Copy link
Contributor Author

Apparently when trying to link to -lhtscodecs it fails because it does not find pthread. 🤔

@teo-tsirpanis
Copy link
Contributor Author

After the latest patch, installing succeeds on a codespace.

@teo-tsirpanis teo-tsirpanis marked this pull request as ready for review July 19, 2024 15:06
@JavierMatosD JavierMatosD merged commit df4590e into microsoft:master Jul 20, 2024
@teo-tsirpanis teo-tsirpanis deleted the htslib branch July 20, 2024 18:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:new-port The issue is requesting a new library to be added; consider making a PR! info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants