rbxl 1.0.0 → 1.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7dfc04eae51753bfa17b28f87476e1da9904efd4748a6a503ef999b58316d419
-  data.tar.gz: dcad9a70d574b225be56d5c942995c5634ef06ac2a882ff030a69d9d1fde1ecb
+  metadata.gz: e9bedc3242085871b368d031e7791aeb925d8d2a53329aebaf1776a0a0d273eb
+  data.tar.gz: e4d6594b3c7d19b63f429b5cb5680df1d4e6e762dd86c2ab33499c97a5389918
 SHA512:
-  metadata.gz: c0c65e0501a613c690795274aee90ee71b26332e916f416e62f7df398d610a67992e230651ffec87a9017243adfb56c517da6498d076e54ed11fe61a8f6dc74d
-  data.tar.gz: fd952f51da370eb1a9a433d661f0c6018a7460e43560b72eb9682cf938a833edda949774ebfcd942add2eabea1d87e37199d993f0b5172153710b0f264d890a9
+  metadata.gz: fac56fdc22b72ff9bf75c3273e8e9a61fbab953c3ddb280618522c121a71a8d530f09792d4c80682b94309b7042604af42b5bfeeab41420e67611cb57d0a57de
+  data.tar.gz: 72f58522b5d7d9a0e1ca16e153a578871e8a2355e12a9066c7f1b1ec1026c72124bdbd7ec72c9e293caf74dfcff0e21386c09db1155c7d2c7549e04ef93abdec

data/CHANGELOG.md

@@ -0,0 +1,11 @@
+# Changelog
+
+## 1.0.1
+
+- Fix ZIP64 handling.
+- Add Go and Rust benchmark comparisons.
+- Align `rbxl/native` with Nokogiri's libxml2 to avoid mixed-library warnings at runtime.
+
+## 1.0.0
+
+- Initial 1.0 release.

data/README.md

@@ -98,6 +98,7 @@ Requirements for the C extension:
 
 ```bash
 bundle install
+cd benchmark && npm install && cd ..
 
 # Run tests (pure Ruby)
 ruby -Ilib -Itest test/rbxl_test.rb
@@ -115,28 +116,42 @@ RBXL_BENCH_WARMUP=1 RBXL_BENCH_ITERATIONS=5 ruby -Ilib benchmark/read_modes.rb
 
 ## Benchmarks
 
-5000 rows x 10 columns, Ruby 3.4 / Python 3.13:
+The performance story is primarily about `rbxl/native`.
+
+`require "rbxl"` remains the portability-first default: no native extension is
+required, the API stays the same, and the fallback path is still useful for
+environments where native builds are inconvenient. But the numbers below are
+best read as:
+
+- `rbxl` = portable baseline
+- `rbxl/native` = performance mode
+
+5000 rows x 10 columns, Ruby 3.4 / Python 3.13 / Node 24:
 
 ![Benchmark chart](benchmark/chart.png)
 
-### Pure Ruby (Nokogiri Reader)
+### Portable Baseline (`require "rbxl"`)
 
 | benchmark | real (s) |
 |---|---|
-| rbxl write | 0.09 |
-| rbxl read | 0.30 |
-| rbxl read values | 0.22 |
-| openpyxl write | 0.36 |
-| openpyxl read | 0.28 |
-| openpyxl read values | 0.26 |
-
-### With `rbxl/native`
-
-| benchmark | real (s) | vs openpyxl |
+| rbxl write | 0.08 |
+| rbxl read | 0.33 |
+| rbxl read values | 0.23 |
+| exceljs write | 0.08 |
+| exceljs read | 0.17 |
+| sheetjs write | 0.13 |
+| sheetjs read | 0.19 |
+| openpyxl write | 0.35 |
+| openpyxl read | 0.22 |
+| openpyxl read values | 0.18 |
+
+### Performance Mode (`require "rbxl/native"`)
+
+| benchmark | real (s) | vs exceljs/openpyxl |
 |---|---|---|
-| rbxl write | **0.04** | 9x faster |
-| rbxl read | **0.08** | 3.5x faster |
-| rbxl read values | **0.03** | 9x faster |
+| rbxl write | **0.04** | about 2x / 9x faster |
+| rbxl read | **0.07** | about 2.6x / 3.2x faster |
+| rbxl read values | **0.03** | about 6.8x faster than openpyxl values |
 
 The comparison script uses these libraries when available:
 
@@ -144,11 +159,16 @@ Benchmark notes:
 
 - `RBXL_BENCH_WARMUP` and `RBXL_BENCH_ITERATIONS` control warmup and repeated runs.
 - Read comparisons use the same `rbxl.xlsx` fixture for `rbxl`, `roo`, `rubyXL`, and `openpyxl`.
+- JS comparisons use the same `rbxl.xlsx` fixture for `exceljs` and `sheetjs`.
 - Write comparisons still measure each library producing its own workbook.
 - `rss_delta_kb` is best-effort process RSS on Linux and should be treated as directional.
+- Install JS benchmark dependencies with `cd benchmark && npm install`.
 
 - `rbxl` for write/read
-- `caxlsx` for write
-- `roo` for read streaming
+- `exceljs` for write/read
+- `sheetjs` for write/read
+- `excelize` (Go) for write/read
+- `rust_xlsxwriter` (Rust) for write
+- `calamine` (Rust) for read
 - `rubyXL` for full workbook read
 - `openpyxl` as a Python reference point when `openpyxl` or `uv` is available

data/ext/rbxl_native/extconf.rb

@@ -1,48 +1,37 @@
 require "mkmf"
 
-# Try to find libxml2 headers and library.
-# Priority:
-#   1. Nokogiri's bundled libxml2 (avoids version mismatch warnings)
-#   2. System pkg-config
-#   3. Common system paths
-#
-# If libxml2 is not available at all, skip compilation gracefully so
-# that `gem install rbxl` never fails — the C extension is optional.
-
-found = false
-
-# 1. Try Nokogiri's bundled libxml2
+# The extension is intentionally built against Nokogiri's vendored libxml2.
+# We only borrow Nokogiri's headers at build time and rely on Nokogiri's
+# extension to export the libxml2 symbols at runtime. Linking against the
+# system libxml2 here would reintroduce mixed-version warnings and can lead
+# to process instability.
+
 begin
-  nokogiri_spec = Gem::Specification.find_by_name("nokogiri")
-  nokogiri_include = File.join(nokogiri_spec.full_gem_path, "ext", "nokogiri", "include", "libxml2")
-  nokogiri_lib = File.join(nokogiri_spec.full_gem_path, "ext", "nokogiri")
-
-  if File.directory?(nokogiri_include) && find_header("libxml/parser.h", nokogiri_include)
-    # Link against Nokogiri's bundled libxml2
-    nokogiri_so = Dir.glob(File.join(nokogiri_lib, "**", "nokogiri.{so,bundle}")).first
-    if nokogiri_so
-      so_dir = File.dirname(nokogiri_so)
-      $LDFLAGS << " -L#{so_dir} -Wl,-rpath,#{so_dir}"
-    end
-    found = have_library("xml2") || true # headers found via Nokogiri, may link at runtime
-  end
-rescue Gem::MissingSpecError
-  # Nokogiri not installed — fall through
+  require "nokogiri"
+rescue LoadError
+  warn "rbxl_native: nokogiri is required to build the C extension"
+  File.write("Makefile", "all install clean:\n\t@:\n")
+  exit 0
 end
 
-# 2. System pkg-config
-found ||= pkg_config("libxml-2.0")
+nokogiri_cppflags = Array(Nokogiri::VERSION_INFO.dig("nokogiri", "cppflags"))
+nokogiri_ldflags = Array(Nokogiri::VERSION_INFO.dig("nokogiri", "ldflags"))
 
-# 3. Common system paths
-found ||= (have_header("libxml/parser.h") && have_library("xml2"))
-found ||= (find_header("libxml/parser.h", "/usr/include/libxml2") && have_library("xml2"))
+$CPPFLAGS = [*nokogiri_cppflags, $CPPFLAGS].reject(&:empty?).join(" ")
+$LDFLAGS = [*nokogiri_ldflags, $LDFLAGS].reject(&:empty?).join(" ")
 
-unless found
-  warn "rbxl_native: libxml2 not found — skipping C extension build"
+unless have_header("libxml/parser.h")
+  warn "rbxl_native: failed to find Nokogiri libxml2 headers"
   File.write("Makefile", "all install clean:\n\t@:\n")
   exit 0
 end
 
+# macOS refuses unresolved references in shared objects unless explicitly told
+# to leave them for runtime lookup in already-loaded extensions like Nokogiri.
+if RUBY_PLATFORM.include?("darwin")
+  append_ldflags("-Wl,-undefined,dynamic_lookup")
+end
+
 # Hardening flags
 $CFLAGS << " -Wall -Wextra -Werror=format-security"
 $CFLAGS << " -D_FORTIFY_SOURCE=2" unless $CFLAGS.include?("_FORTIFY_SOURCE")

data/lib/rbxl/native.rb

@@ -1,3 +1,5 @@
+require "nokogiri"
+
 begin
   require "rbxl_native/rbxl_native"
 rescue LoadError

data/lib/rbxl/version.rb

@@ -1,3 +1,3 @@
 module Rbxl
-  VERSION = "1.0.0"
+  VERSION = "1.0.1"
 end

data/lib/rbxl/write_only_workbook.rb

@@ -20,16 +20,23 @@ module Rbxl
       ensure_writable!
       raise Error, "at least one worksheet is required" if worksheets.empty?
 
-      Zip::OutputStream.open(path) do |zip|
-        write_entry(zip, "[Content_Types].xml", content_types_xml)
-        write_entry(zip, "_rels/.rels", root_rels_xml)
-        write_entry(zip, "xl/workbook.xml", workbook_xml)
-        write_entry(zip, "xl/_rels/workbook.xml.rels", workbook_rels_xml)
-        write_entry(zip, "xl/styles.xml", styles_xml)
-
-        worksheets.each_with_index do |sheet, index|
-          write_entry(zip, "xl/worksheets/sheet#{index + 1}.xml", sheet.to_xml)
+      previous_zip64 = Zip.write_zip64_support
+      begin
+        Zip.write_zip64_support = false
+
+        Zip::OutputStream.open(path) do |zip|
+          write_entry(zip, "[Content_Types].xml", content_types_xml)
+          write_entry(zip, "_rels/.rels", root_rels_xml)
+          write_entry(zip, "xl/workbook.xml", workbook_xml)
+          write_entry(zip, "xl/_rels/workbook.xml.rels", workbook_rels_xml)
+          write_entry(zip, "xl/styles.xml", styles_xml)
+
+          worksheets.each_with_index do |sheet, index|
+            write_entry(zip, "xl/worksheets/sheet#{index + 1}.xml", sheet.to_xml)
+          end
         end
+      ensure
+        Zip.write_zip64_support = previous_zip64
       end
 
       @saved = true

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rbxl
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.0.1
 platform: ruby
 authors:
 - Taro KOBAYASHI
@@ -51,6 +51,7 @@ extensions:
 - ext/rbxl_native/extconf.rb
 extra_rdoc_files: []
 files:
+- CHANGELOG.md
 - LICENSE.txt
 - README.md
 - Rakefile