Subject: [bug#50054] [PATCH v2 3/3] gnu: Add bgfx.

From: Ludovic Courtès <[email-masked]>
Date: Mon, 13 May 2024 14:40:12 +0200

Andy Tai <[email-masked]> skribis:

> * gnu/packages/graphics.scm (bgfx): New variable.
>
> Change-Id: I2737803c838229ed04fbfe30d0e67749444d871b

[...]

> +      (inputs `(("bx-sources" ,(package-source bx))
> +                ("bimg-sources" ,(package-source bimg))
> +                ("mesa" ,mesa)
> +                ("mesa-headers" ,mesa-headers)
> +                ("vulkan-headers" ,vulkan-headers)))

Is it not possible to use pre-built copies of Bx and Bimg?

If not, then please change the above to:

  (inputs (list mesa mesa-headers vulkan-headers))

and refer to the source directly from the gexp, like so:

  #:phases #~(…  #$(package-source bx) …)

> +      (home-page "https://bkaradzic.github.io/bgfx/overview.html")
> +      (synopsis "Cross-platform rendering library")
> +      (description
> +       "bgfx is a cross-platform, graphics API agnostic,
> +Bring Your Own Engine/Framework' style rendering library supporting Direct3D,
> +OpenGL/OpenGL ES, Metal, Vulkan and WebGL.")

Would be great if you could expound a bit, especially the synopsis.

Ludo’.