fix: don't add library name config unnecessarily

Author: robbielyman

build.zig

@@ -16,7 +16,7 @@ pub fn build(b: *Build) void {
     const optimize = b.standardOptimizeOption(.{});
 
     const lang = b.option(Language, "lang", "Lua language version to build") orelse .lua54;
-    const library_name = b.option([]const u8, "library_name", "Library name for lua linking, default is `lua`") orelse null;
+    const library_name = b.option([]const u8, "library_name", "Library name for lua linking, default is `lua`") orelse "lua";
     const shared = b.option(bool, "shared", "Build shared library instead of static") orelse false;
     const luau_use_4_vector = b.option(bool, "luau_use_4_vector", "Build Luau to use 4-vectors instead of the default 3-vector.") orelse false;
 
@@ -32,7 +32,6 @@ pub fn build(b: *Build) void {
     // Expose build configuration to the ziglua module
     const config = b.addOptions();
     config.addOption(Language, "lang", lang);
-    config.addOption(?[]const u8, "library_name", library_name);
     config.addOption(bool, "luau_use_4_vector", luau_use_4_vector);
     zlua.addOptions("config", config);
 

build/lua.zig

@@ -12,7 +12,7 @@ pub const Language = enum {
     luau,
 };
 
-pub fn configure(b: *Build, target: Build.ResolvedTarget, optimize: std.builtin.OptimizeMode, upstream: *Build.Dependency, lang: Language, shared: bool, library_name: ?[]const u8) *Step.Compile {
+pub fn configure(b: *Build, target: Build.ResolvedTarget, optimize: std.builtin.OptimizeMode, upstream: *Build.Dependency, lang: Language, shared: bool, library_name: []const u8) *Step.Compile {
     const version: std.SemanticVersion = switch (lang) {
         .lua51 => .{ .major = 5, .minor = 1, .patch = 5 },
         .lua52 => .{ .major = 5, .minor = 2, .patch = 4 },
@@ -23,14 +23,14 @@ pub fn configure(b: *Build, target: Build.ResolvedTarget, optimize: std.builtin.
 
     const lib = if (shared)
         b.addSharedLibrary(.{
-            .name = library_name orelse "lua",
+            .name = library_name,
             .target = target,
             .optimize = optimize,
             .version = version,
         })
     else
         b.addStaticLibrary(.{
-            .name = library_name orelse "lua",
+            .name = library_name,
             .target = target,
             .optimize = optimize,
             .version = version,