Augment error message when opening files for external tools

crates/bridge_core/src/lib.rs

@@ -14,6 +14,8 @@
 //!
 //! [cbindgen]: https://github.com/eqrion/cbindgen
 //!
+//! If you change the interfaces here, rerun cbindgen as described in the README!
+//!
 //! In order to provide access to a C/C++ engine in Rust, you should create some
 //! kind of interface that expects to be given a reference to a
 //! [`CoreBridgeLauncher`] struct. You should use that struct's
@@ -589,7 +591,7 @@ impl<'a> CoreBridgeState<'a> {
         }
     }
 
-    fn input_get_mtime(&mut self, handle: *mut InputHandle) -> libc::time_t {
+    fn input_get_mtime(&mut self, handle: *mut InputHandle) -> i64 {
         let rhandle: &mut InputHandle = unsafe { &mut *handle };
 
         let maybe_time = match rhandle.get_unix_mtime() {
@@ -601,7 +603,7 @@ impl<'a> CoreBridgeState<'a> {
         };
 
         if let Some(t) = maybe_time {
-            t as libc::time_t
+            t
         } else {
             1 // Intentionally make this distinguishable from the error value 0
         }
@@ -1020,10 +1022,7 @@ pub extern "C" fn ttbc_input_get_size(
 
 /// Get the modification time of a Tectonic input file.
 #[no_mangle]
-pub extern "C" fn ttbc_input_get_mtime(
-    es: &mut CoreBridgeState,
-    handle: *mut InputHandle,
-) -> libc::time_t {
+pub extern "C" fn ttbc_input_get_mtime(es: &mut CoreBridgeState, handle: *mut InputHandle) -> i64 {
     es.input_get_mtime(handle)
 }
 

crates/bridge_core/support/support.c

@@ -254,7 +254,12 @@ ttstub_input_get_size(rust_input_handle_t handle)
 time_t
 ttstub_input_get_mtime(rust_input_handle_t handle)
 {
-    return ttbc_input_get_mtime(tectonic_global_bridge_core, handle);
+    /* Due to the Musl 1.2 "time64" transition, we can't safely bridge time_t
+     * between Rust and C code. And formally, ISO C provides nearly no
+     * guarantees about what the type time_t actually is. So let's just cast and
+     * hope for the best. */
+    int64_t ti = ttbc_input_get_mtime(tectonic_global_bridge_core, handle);
+    return (time_t) ti;
 }
 
 

crates/bridge_core/support/tectonic_bridge_core_generated.h

@@ -262,7 +262,7 @@ size_t ttbc_input_get_size(ttbc_state_t *es, ttbc_input_handle_t *handle);
 /**
  * Get the modification time of a Tectonic input file.
  */
-time_t ttbc_input_get_mtime(ttbc_state_t *es, ttbc_input_handle_t *handle);
+int64_t ttbc_input_get_mtime(ttbc_state_t *es, ttbc_input_handle_t *handle);
 
 /**
  * Seek in a Tectonic input stream.

crates/bridge_flate/src/lib.rs

@@ -6,6 +6,8 @@
 //! This crate provides a few extern "C" functions that expose the functionality
 //! of the flate2 crate in a C API that can be consumed by other C/C++ code in
 //! the Tectonic codebase.
+//!
+//! If you change the interfaces here, rerun cbindgen as described in the README!
 
 use flate2::{Compress, Compression, Decompress, FlushCompress, FlushDecompress, Status};
 use std::{

crates/engine_bibtex/src/lib.rs

@@ -16,6 +16,8 @@
 //!
 //! [Tectonic]: https://tectonic-typesetting.github.io/
 //! [`tectonic`]: https://docs.rs/tectonic/
+//!
+//! If you change the interfaces here, rerun cbindgen as described in the README!
 
 use std::ffi::CString;
 use tectonic_bridge_core::{CoreBridgeLauncher, EngineAbortedError};

crates/engine_xdvipdfmx/src/lib.rs

@@ -124,7 +124,7 @@ impl XdvipdfmxEngine {
                 .build_date
                 .duration_since(SystemTime::UNIX_EPOCH)
                 .expect("invalid build date")
-                .as_secs() as libc::time_t,
+                .as_secs(),
         };
 
         let cdvi = CString::new(dvi)?;
@@ -147,6 +147,8 @@ impl XdvipdfmxEngine {
 
 #[doc(hidden)]
 pub mod c_api {
+    // If you change the interfaces here, rerun cbindgen as described in the README!
+
     use tectonic_bridge_core::CoreBridgeState;
 
     #[derive(Debug)]
@@ -155,7 +157,7 @@ pub mod c_api {
         pub paperspec: *const libc::c_char,
         pub enable_compression: libc::c_uchar,
         pub deterministic_tags: libc::c_uchar,
-        pub build_date: libc::time_t,
+        pub build_date: u64,
     }
 
     #[allow(improper_ctypes)] // for CoreBridgeState

crates/engine_xdvipdfmx/xdvipdfmx/dvipdfmx.c

@@ -499,6 +499,9 @@ tt_engine_xdvipdfmx_main(
     return 99;
   }
 
+  /* See ttstub_input_get_mtime() in tectonic_bridge_core about bridging time_t
+   * over FFI. */
+
   rv = dvipdfmx_main(
     pdfname,
     dviname,
@@ -509,7 +512,7 @@ tt_engine_xdvipdfmx_main(
     (bool) config->deterministic_tags,
     false, /* quiet */
     0, /* verbose */
-    config->build_date,
+    (time_t) config->build_date,
     config->paperspec
   );
 

crates/engine_xdvipdfmx/xdvipdfmx/xdvipdfmx_bindings.h

@@ -10,7 +10,7 @@ typedef struct {
   const char *paperspec;
   unsigned char enable_compression;
   unsigned char deterministic_tags;
-  time_t build_date;
+  uint64_t build_date;
 } XdvipdfmxConfig;
 
 #ifdef __cplusplus

crates/engine_xetex/src/lib.rs

@@ -228,7 +228,7 @@ impl TexEngine {
                     self.build_date
                         .duration_since(SystemTime::UNIX_EPOCH)
                         .expect("invalid build date")
-                        .as_secs() as libc::time_t,
+                        .as_secs(),
                 )
             };
 
@@ -245,6 +245,8 @@ impl TexEngine {
 
 #[doc(hidden)]
 pub mod c_api {
+    // If you change the interfaces here, rerun cbindgen as described in the README!
+
     use tectonic_bridge_core::CoreBridgeState;
 
     #[allow(improper_ctypes)] // for CoreBridgeState
@@ -258,7 +260,7 @@ pub mod c_api {
             api: &mut CoreBridgeState,
             dump_name: *const libc::c_char,
             input_file_name: *const libc::c_char,
-            build_date: libc::time_t,
+            build_date: u64,
         ) -> libc::c_int;
     }
 }

crates/engine_xetex/xetex/xetex-engine-interface.c

@@ -15,7 +15,7 @@ int tt_engine_xetex_main(
     ttbc_state_t *api,
     const char *dump_name,
     const char *input_file_name,
-    time_t build_date
+    uint64_t build_date
 );
 
 /* These functions aren't used within the C/C++ library, but are called
@@ -53,7 +53,7 @@ tt_engine_xetex_main(
     ttbc_state_t *api,
     const char *dump_name,
     const char *input_file_name,
-    time_t build_date
+    uint64_t build_date
 ) {
     int rv;
 
@@ -62,7 +62,9 @@ tt_engine_xetex_main(
         return HISTORY_FATAL_ERROR;
     }
 
-    rv = tt_run_engine(dump_name, input_file_name, build_date);
+    /* See ttstub_input_get_mtime() in tectonic_bridge_core about bridging time_t
+    * over FFI. */
+    rv = tt_run_engine(dump_name, input_file_name, (time_t) build_date);
     ttbc_global_engine_exit();
     return rv;
 }

crates/engine_xetex/xetex/xetex_bindings.h

@@ -27,7 +27,7 @@ extern int tt_xetex_set_int_variable(const char *var_name, int value);
 extern int tt_engine_xetex_main(ttbc_state_t *api,
                                 const char *dump_name,
                                 const char *input_file_name,
-                                time_t build_date);
+                                uint64_t build_date);
 
 #ifdef __cplusplus
 } // extern "C"

crates/pdf_io/pdf_io/dpx-pdfobj.c

@@ -675,11 +675,11 @@ pdf_out_flush (void)
 
         if (dpx_conf.verbose_level > 0) {
             if (p->options.compression.level > 0) {
-                dpx_message("Compression saved %ld bytes\n", p->output.compression_saved);
+                dpx_message("Compression saved %"PRIuZ" bytes\n", p->output.compression_saved);
             }
         }
 
-        dpx_message("%ld bytes written", p->output.file_position);
+        dpx_message("%"PRIuZ" bytes written", p->output.file_position);
 
         ttstub_output_close(p->output.handle);
         p->output.handle = NULL;

crates/xetex_format/src/enums/dynamic.rs

@@ -153,7 +153,7 @@ macro_rules! inner_declare_primitive_value {
 ///
 /// Usage:
 ///
-/// ```
+/// ```text
 /// {
 ///     var $VARNAME1 $SYMNAME1 $SINCE1 [$VALUESOURCE1],
 ///     var $VARNAME2 $SYMNAME2 $SINCE2 [$VALUESOURCE2],
@@ -163,7 +163,7 @@ macro_rules! inner_declare_primitive_value {
 ///
 /// basically becomes
 ///
-/// ```
+/// ```text
 /// pub enum Enumeration {
 ///     $VARNAME1,
 ///     $VARNAME2,
@@ -207,14 +207,14 @@ macro_rules! inner_declare_enum {
 ///
 /// Usage:
 ///
-/// ```
+/// ```text
 /// (var $VARNAME $SYMNAME $SINCE [$VALUESOURCE])
 /// ```
 ///
 /// becomes a `DynamicEnumItemDescription` initializer with `absval`
 /// of `Some(Enumeration::$VARNAME)`, while
 ///
-/// ```
+/// ```text
 /// (not ....)
 /// ```
 ///
@@ -259,7 +259,7 @@ macro_rules! inner_declare_desc {
 ///
 /// Usage:
 ///
-/// ```
+/// ```text
 /// declare! {
 ///     EnumName enum_name {
 ///         var First  FIRST_CODE     10 [Next],
@@ -271,7 +271,7 @@ macro_rules! inner_declare_desc {
 ///
 /// evaluates to something along the lines of
 ///
-/// ```
+/// ```text
 /// pub mod enum_name {
 ///     pub enum Enumeration {
 ///         First,

crates/xetex_layout/layout/xetex-XeTeXFontMgr_FC.cpp

@@ -122,7 +122,8 @@ XeTeXFontMgr_FC::readNames(FcPattern* pat)
                 case kPreferredSubfamilyName:
                     {
                         bool preferredName = false;
-                        if (nameRec.platform_id == TT_PLATFORM_MACINTOSH
+                        /* Tectonic: macRomanConv may not be available; see comment below */
+                        if (macRomanConv != NULL && nameRec.platform_id == TT_PLATFORM_MACINTOSH
                                 && nameRec.encoding_id == TT_MAC_ID_ROMAN && nameRec.language_id == 0) {
                             utf8name = convertToUtf8(macRomanConv, nameRec.string, nameRec.string_len);
                             preferredName = true;
@@ -319,10 +320,21 @@ XeTeXFontMgr_FC::initialize()
         _tt_abort("FreeType initialization failed");
 
     UErrorCode err = U_ZERO_ERROR;
+
+    /* Tectonic: Alpine >=3.16 splits the ICU data in a way that seems to make
+     * the "macintosh" converter unavailable in our MUSL static builds. I don't
+     * see a workaround: installing `icu-data-full` doesn't help, I think
+     * because the static libraries don't access the data file that it provides.
+     */
     macRomanConv = ucnv_open("macintosh", &err);
+    if (!U_SUCCESS(err)) {
+        err = U_ZERO_ERROR;
+        macRomanConv = NULL;
+    }
+
     utf16beConv = ucnv_open("UTF16BE", &err);
     utf8Conv = ucnv_open("UTF8", &err);
-    if (err)
+    if (!U_SUCCESS(err))
         _tt_abort("cannot read font names");
 
     FcPattern* pat = FcNameParse((const FcChar8*)":outline=true");

src/driver.rs

@@ -338,7 +338,10 @@ impl BridgeState {
                     ));
                 }
 
-                let mut ih = self.input_open_name(name, status).must_exist()?;
+                let mut ih = ctry!(
+                    self.input_open_name(name, status).must_exist();
+                    "can't open path `{}`", name
+                );
 
                 // If the input path is absolute, we don't need to create a
                 // version in the tempdir, and in fact the current