Skip to content

Workflow fix#11

Closed
cmaceves wants to merge 85 commits intogkarthik:masterfrom
andersen-lab:workflow_fix
Closed

Workflow fix#11
cmaceves wants to merge 85 commits intogkarthik:masterfrom
andersen-lab:workflow_fix

Conversation

@cmaceves
Copy link
Contributor

@cmaceves cmaceves commented Aug 2, 2025

Bumping Ubuntu version from 20.04 to 22.04 in workflows.

cmaceves and others added 30 commits February 3, 2023 10:52
Bump version number to 1.3.2
changing type of min_depth from uint8_t to uint32_t
… trim_io

� Conflicts:
�	src/trim_primer_quality.cpp
Updating for the acceptance of stdin
cmaceves and others added 29 commits June 1, 2023 08:40
Update Dockerfile htslib version and ubuntu version
Attempting to build ivar up to version 1.4.1 with Gcc 13 will result
with the following errors:

	primer_bed.h:13:3: error: ‘uint32_t’ does not name a type
	   13 |   uint32_t start;  // 0 based
	      |   ^~~~~~~~
	primer_bed.h:6:1: note: ‘uint32_t’ is defined in header ‘<cstdint>’; did you forget to ‘#include <cstdint>’?
	    5 | #include <vector>
	  +++ |+#include <cstdint>
	    6 |
	[…]
	parse_gff.h:47:3: error: ‘uint64_t’ does not name a type
	   47 |   uint64_t get_start();
	      |   ^~~~~~~~
	parse_gff.h:6:1: note: ‘uint64_t’ is defined in header ‘<cstdint>’; did you forget to ‘#include <cstdint>’?
	    5 | #include <regex>
	  +++ |+#include <cstdint>
	    6 | #include <sstream>

Per the hints, this patch will resolve the build problem with the
newer compiler version.  This problem has been initially reported in
[Debian Bug#1037700].

[Debian Bug#1037700]: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1037700

Signed-off-by: Étienne Mollier <emollier@debian.org>
# Conflicts:
#	tests/Makefile.am
* Install autoconf with brew.
* Bumped up htslib version for macos.
fix build failure with gcc-13
@cmaceves cmaceves closed this Aug 2, 2025
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.

6 participants