Bump liberty, replace help2man with help2adoc

CMakeLists.txt

@@ -34,49 +34,7 @@ endif ()
 
 # Dependencies
 set (CMAKE_MODULE_PATH ${PROJECT_SOURCE_DIR}/liberty/cmake)
-
-# TODO(p): Shove this into IconUtils.cmake.
-function (icon_to_iconset_size name svg size iconset outputs)
-	math (EXPR _size2x "${size} * 2")
-	set (_dimensions "${size}x${size}")
-	set (_png1x "${iconset}/icon_${_dimensions}.png")
-	set (_png2x "${iconset}/icon_${_dimensions}@2x.png")
-	set (${outputs} "${_png1x};${_png2x}" PARENT_SCOPE)
-
-	set (_find_program_REQUIRE)
-	if (NOT ${CMAKE_VERSION} VERSION_LESS 3.18.0)
-		set (_find_program_REQUIRE REQUIRED)
-	endif ()
-
-	find_program (rsvg_convert_EXECUTABLE rsvg-convert ${_find_program_REQUIRE})
-	add_custom_command (OUTPUT "${_png1x}" "${_png2x}"
-		COMMAND ${CMAKE_COMMAND} -E make_directory "${iconset}"
-		COMMAND ${rsvg_convert_EXECUTABLE} "--output=${_png1x}"
-			"--width=${size}" "--height=${size}" -- "${svg}"
-		COMMAND ${rsvg_convert_EXECUTABLE} "--output=${_png2x}"
-			"--width=${_size2x}" "--height=${_size2x}" -- "${svg}"
-		DEPENDS "${svg}"
-		COMMENT "Generating ${name} ${_dimensions} icons" VERBATIM)
-endfunction ()
-function (icon_to_icns svg output_basename output)
-	get_filename_component (_name "${output_basename}" NAME_WE)
-	set (_iconset "${PROJECT_BINARY_DIR}/${_name}.iconset")
-	set (_icon "${PROJECT_BINARY_DIR}/${output_basename}")
-	set (${output} "${_icon}" PARENT_SCOPE)
-
-	set (_icon_png_list)
-	foreach (_icon_size 16 32 128 256 512)
-		icon_to_iconset_size ("${_name}" "${svg}"
-			"${_icon_size}" "${_iconset}" _icon_pngs)
-		list (APPEND _icon_png_list ${_icon_pngs})
-	endforeach ()
-	add_custom_command (OUTPUT "${_icon}"
-		COMMAND iconutil -c icns -o "${_icon}" "${_iconset}"
-		DEPENDS ${_icon_png_list}
-		COMMENT "Generating ${_name} icon" VERBATIM)
-	set_source_files_properties ("${_icon}" PROPERTIES
-		MACOSX_PACKAGE_LOCATION Resources)
-endfunction ()
+include (IconUtils)
 
 find_package (PkgConfig REQUIRED)
 pkg_check_modules (libusb libusb-1.0)
@@ -138,7 +96,6 @@ endif ()
 if (WITH_HIDAPI AND WIN32)
 	list (APPEND targets_gui eizoctltray)
 
-	include (IconUtils)
 	set (icon_png_list)
 	foreach (icon_size 16 32 48)
 		icon_to_png (eizoctltray ${PROJECT_SOURCE_DIR}/eizoctltray.svg
@@ -181,44 +138,57 @@ endif ()
 
 # Generate documentation from help output
 if (NOT WIN32 AND NOT CMAKE_CROSSCOMPILING)
-	find_program (HELP2MAN_EXECUTABLE help2man)
-	if (NOT HELP2MAN_EXECUTABLE)
-		message (FATAL_ERROR "help2man not found")
-	endif ()
+	set (HELP2ADOC "${PROJECT_SOURCE_DIR}/liberty/tools/help2adoc.awk")
+	set (ASCIIMAN "${PROJECT_SOURCE_DIR}/liberty/tools/asciiman.awk")
 
 	foreach (target ${targets})
-		set (page_output "${PROJECT_BINARY_DIR}/${target}.1")
-		list (APPEND project_MAN_PAGES "${page_output}")
-		add_custom_command (OUTPUT ${page_output}
-			COMMAND ${HELP2MAN_EXECUTABLE} -N
-				"${PROJECT_BINARY_DIR}/${target}" -o ${page_output}
-			DEPENDS ${target}
-			COMMENT "Generating man page for ${target}" VERBATIM)
+		set (page_adoc "${PROJECT_BINARY_DIR}/${target}.1.adoc")
+		set (page_roff "${PROJECT_BINARY_DIR}/${target}.1")
+		list (APPEND project_MAN_PAGES "${page_roff}")
+
+		# $<TARGET_FILE:tgt> could be used, if we didn't have to escape it.
+		string (REPLACE "\\" "\\\\"
+			target_path "${PROJECT_BINARY_DIR}/${target}")
+		add_custom_command (OUTPUT "${page_adoc}"
+			COMMAND env LC_ALL=C awk -f "${HELP2ADOC}"
+				-v "Target=${target_path}" > "${page_adoc}"
+			DEPENDS "${target}" "${HELP2ADOC}"
+			COMMENT "Generating AsciiDoc man page for ${target}" VERBATIM)
+		add_custom_command (OUTPUT "${page_roff}"
+			COMMAND env LC_ALL=C awk -f "${ASCIIMAN}"
+				"${page_adoc}" > "${page_roff}"
+			DEPENDS "${page_adoc}" "${ASCIIMAN}"
+			COMMENT "Generating roff man page for ${target}" VERBATIM)
 	endforeach ()
 
 	add_custom_target (docs ALL DEPENDS ${project_MAN_PAGES})
 endif ()
 
 # The files to be installed
-include (GNUInstallDirs)
+if (NOT WIN32)
+	include (GNUInstallDirs)
 
-# These should be accessible by users, but need to touch system devices.
-# Use the setuid bit, for simplicity.
-set (SETUID "SETUID" CACHE STRING "Set this empty on permission issues")
-install (TARGETS ${targets} DESTINATION ${CMAKE_INSTALL_BINDIR}
-	PERMISSIONS
-		OWNER_WRITE OWNER_READ OWNER_EXECUTE
-		GROUP_READ GROUP_EXECUTE
-		WORLD_READ WORLD_EXECUTE
-		${SETUID})
-install (TARGETS ${targets_gui} DESTINATION ${CMAKE_INSTALL_BINDIR})
-install (FILES LICENSE DESTINATION ${CMAKE_INSTALL_DOCDIR})
+	# These should be accessible by users, but need to touch system devices.
+	# Use the setuid bit, for simplicity.
+	set (SETUID "SETUID" CACHE STRING "Set this empty on permission issues")
+	install (TARGETS ${targets} DESTINATION ${CMAKE_INSTALL_BINDIR}
+		PERMISSIONS
+			OWNER_WRITE OWNER_READ OWNER_EXECUTE
+			GROUP_READ GROUP_EXECUTE
+			WORLD_READ WORLD_EXECUTE
+			${SETUID})
+	install (TARGETS ${targets_gui} DESTINATION ${CMAKE_INSTALL_BINDIR})
+	install (FILES LICENSE DESTINATION ${CMAKE_INSTALL_DOCDIR})
+	foreach (page ${project_MAN_PAGES})
+		string (REGEX MATCH "\\.([0-9])$" manpage_suffix "${page}")
+		install (FILES "${page}"
+			DESTINATION "${CMAKE_INSTALL_MANDIR}/man${CMAKE_MATCH_1}")
+	endforeach ()
 
-foreach (page ${project_MAN_PAGES})
-	string (REGEX MATCH "\\.([0-9])$" manpage_suffix "${page}")
-	install (FILES "${page}"
-		DESTINATION "${CMAKE_INSTALL_MANDIR}/man${CMAKE_MATCH_1}")
-endforeach ()
+	set (CPACK_SET_DESTDIR TRUE)
+else ()
+	install (TARGETS ${targets} ${targets_gui} DESTINATION .)
+endif ()
 
 # CPack
 set (CPACK_PACKAGE_VENDOR "Premysl Eric Janouch")
@@ -232,5 +202,4 @@ set (CPACK_SOURCE_GENERATOR "TGZ;ZIP")
 set (CPACK_SOURCE_IGNORE_FILES "/\\\\.git;/build;/CMakeLists.txt.user")
 set (CPACK_SOURCE_PACKAGE_FILE_NAME "${PROJECT_NAME}-${PROJECT_VERSION}")
 
-set (CPACK_SET_DESTDIR TRUE)
 include (CPack)

README.adoc

@@ -59,7 +59,7 @@ https://git.janouch.name/p/usb-drivers/releases[the Releases page on Gitea].
 Building
 --------
 Build dependencies:
- CMake, pkg-config, liberty (included), help2man +
+ CMake, pkg-config, liberty (included) +
 Runtime dependencies:
  libusb-1.0 (elksmart-comm, razer-bw-te-ctl), hidapi >= 0.14 (eizoctl)
 

eizo-pcap-decode.go

@@ -73,10 +73,17 @@ var (
 	fmtIn, fmtOut, fmtReset string
 )
 
-func decodeSubreport(data []byte) string {
-	if len(data) < 6 {
+func decodeSubreport(id byte, data []byte) string {
+	critical := isCriticalSubreport(id)
+	if len(data) < 6 || critical && len(data) < 8 {
 		return fmt.Sprintf("%x", data)
 	}
+
+	var cs uint16
+	if critical {
+		data, cs = data[2:], le.Uint16(data[0:2])
+	}
+
 	usage := uint32(le.Uint16(data[:2]))<<16 | uint32(le.Uint16(data[2:4]))
 	filtered := make([]byte, len(data)-6)
 	for i, b := range data[6:] {
@@ -86,8 +93,17 @@ func decodeSubreport(data []byte) string {
 			filtered[i] = b
 		}
 	}
-	return fmt.Sprintf("<> %08x %04x %x %s", usage, le.Uint16(data[4:6]),
-		data[6:], string(filtered))
+	if critical {
+		return fmt.Sprintf("<> %08x %04x=%04x %x %s",
+			usage, cs, le.Uint16(data[4:6]), data[6:], string(filtered))
+	} else if usage == 0xff0000f1 {
+		// No idea what this is, but it follows the format.
+		return fmt.Sprintf("<> %08x %04x %s",
+			usage, le.Uint16(data[4:6]), decodeMP(data[6:]))
+	} else {
+		return fmt.Sprintf("<> %08x %04x %x %s",
+			usage, le.Uint16(data[4:6]), data[6:], string(filtered))
+	}
 }
 
 func decodeResult(data []byte) string {
@@ -131,6 +147,14 @@ func isGetSubreport(id byte) bool {
 	return false
 }
 
+func isCriticalSubreport(id byte) bool {
+	switch id {
+	case 11, 12, 13, 14:
+		return true
+	}
+	return false
+}
+
 func isSubreport(id byte) bool {
 	return isSetSubreport(id) || isGetSubreport(id)
 }
@@ -143,7 +167,7 @@ func processInterrupt(p *Packet) {
 	if *raw {
 		fmt.Printf("%s INT %02x %x\n", p.addr(), data[0], data[1:])
 	} else if isSubreport(data[0]) {
-		fmt.Printf("%s INT %s\n", p.addr(), decodeSubreport(data[1:]))
+		fmt.Printf("%s INT %s\n", p.addr(), decodeSubreport(data[0], data[1:]))
 	}
 }
 
@@ -163,7 +187,7 @@ func processControl(p *Packet) {
 			fmt.Printf("%s IN  SR %x\n", p.addr(), data[5:])
 		} else if isGetSubreport(data[0]) {
 			fmt.Printf("%s IN  %s%s%s\n", p.addr(),
-				fmtIn, decodeSubreport(data[1:]), fmtReset)
+				fmtIn, decodeSubreport(data[0], data[1:]), fmtReset)
 		} else if data[0] == 6 {
 			fmt.Printf("%s IN  PC %04x\n", p.addr(), le.Uint16(data[1:]))
 		} else if data[0] == 7 {
@@ -172,6 +196,8 @@ func processControl(p *Packet) {
 			fmt.Printf("%s IN  ID %s %s\n", p.addr(), data[1:9], data[9:])
 		} else if data[0] == 9 {
 			fmt.Printf("%s IN  MP %s\n", p.addr(), decodeMP(data[1:]))
+		} else if data[0] == 10 {
+			fmt.Printf("%s IN  CS %04x\n", p.addr(), le.Uint16(data[1:]))
 		} else {
 			fmt.Printf("%s IN  %02x %x\n", p.addr(), data[0], data[1:])
 		}
@@ -180,7 +206,9 @@ func processControl(p *Packet) {
 			fmt.Printf("%s OUT %02x %x\n", p.addr(), data[0], data[1:])
 		} else if isSetSubreport(data[0]) {
 			fmt.Printf("%s OUT %s%s%s\n", p.addr(),
-				fmtOut, decodeSubreport(data[1:]), fmtReset)
+				fmtOut, decodeSubreport(data[0], data[1:]), fmtReset)
+		} else if data[0] == 10 {
+			fmt.Printf("%s OUT CS %04x\n", p.addr(), le.Uint16(data[1:]))
 		} else if data[0] != 1 && !isGetSubreport(data[0]) {
 			fmt.Printf("%s OUT %02x %x\n", p.addr(), data[0], data[1:])
 		}

elksmart-comm.c

@@ -196,7 +196,7 @@ compress_value(unsigned value, struct str *encoded)
 }
 
 static void
-compress_pulses (const struct pulse *pulses, size_t len, struct str *encoded)
+compress_pulses(const struct pulse *pulses, size_t len, struct str *encoded)
 {
 	unsigned counts[len];
 	memset(counts, 0, sizeof counts);
@@ -527,15 +527,15 @@ send_identify(libusb_device_handle *device, struct error **e)
 #if 0
 	// The EKX4S does not respond to this request.
 	static uint8_t c_serial[] = { -5, -5, -5, -5 };
-	if ((result = libusb_bulk_transfer (device, g.endpoint_out,
+	if ((result = libusb_bulk_transfer(device, g.endpoint_out,
 		c_serial, sizeof c_serial, &len, 100)))
-		return error_set (e, "serial/send: %s", libusb_strerror (result));
-	if ((result = libusb_bulk_transfer (device, g.endpoint_in,
+		return error_set(e, "serial/send: %s", libusb_strerror(result));
+	if ((result = libusb_bulk_transfer(device, g.endpoint_in,
 		buffer, sizeof buffer, &len, 100)))
-		return error_set (e, "serial/recv: %s", libusb_strerror (result));
+		return error_set(e, "serial/recv: %s", libusb_strerror(result));
 	if (len < (int) sizeof c_serial ||
-		memcmp (buffer, c_serial, sizeof c_serial))
-		return error_set (e, "serial retrieval failed");
+		memcmp(buffer, c_serial, sizeof c_serial))
+		return error_set(e, "serial retrieval failed");
 #endif
 	return true;
 }

razer-bw-te-ctl.c

@@ -198,8 +198,8 @@ parse_options(int argc, char *argv[],
 	};
 
 	if (argc == 1) {
-		show_usage (argv[0]);
-		exit (EXIT_FAILURE);
+		show_usage(argv[0]);
+		exit(EXIT_FAILURE);
 	}
 
 	int c;
@@ -256,7 +256,7 @@ parse_options(int argc, char *argv[],
 			!strcasecmp(optarg, "on") ||
 			!strcasecmp(optarg, "yes")) {
 			new_config->gaming_mode = true;
-		} else if (!strcasecmp (optarg, "false") ||
+		} else if (!strcasecmp(optarg, "false") ||
 			!strcasecmp(optarg, "off") ||
 			!strcasecmp(optarg, "no")) {
 			new_config->gaming_mode = false;
@@ -362,7 +362,7 @@ main(int argc, char *argv[])
 
 	if ((result = apply_options(device, &options, &new_config)))
 		FAIL(error_4, "operation failed: %s\n",
-			libusb_error_name (result));
+			libusb_error_name(result));
 error_4:
 	if ((result = libusb_release_interface(device, BW_CTL_IFACE)))
 		FAIL(error_3, "couldn't release interface: %s\n",