Dear mentors,

I am looking for a sponsor for my package "scenefx":

* Package name : scenefx
Version : 0.5-1
Upstream Author : wlrfx
* URL : https://github.com/wlrfx/scenefx (https://github.com/wlrfx/scenefx)
* License : MIT (tinywl/ example subdirectory: CC0-1.0)
Section : libs

This closes the ITP bug: https://bugs.debian.org/1143103 (https://bugs.debian.org/1143103)

It builds those binary packages:

libscenefx-0.5-0 - wlroots scene API with eye-candy effects (shared library)
libscenefx-0.5-dev - wlroots scene API with eye-candy effects (development files)

To access further information about this package, please visit the
following URL:

https://salsa.debian.org/damian/scenefx (https://salsa.debian.org/damian/scenefx)

Alternatively, one can download the package with dget using this command:

dget -x https://mentors.debian.net/debian/pool/main/s/scenefx/scenefx_0.5-1.dsc (https://mentors.debian.net/debian/pool/main/s/scenefx/scenefx_0.5-1.dsc)

More information about scenefx can be obtained from
https://github.com/wlrfx/scenefx (https://github.com/wlrfx/scenefx).

Changes since the last upload:

Initial release. (Closes: #1143103)

scenefx is a small, actively developed wlroots-ecosystem library (part of
the wlrfx project, alongside dwl and owl) providing visual effects for
wlroots-based Wayland compositors. It currently has no packaging in any
major distribution's official repositories. I noticed an earlier, apparently unfinished packaging attempt on
mentors.debian.net under a different uploader; there is no corresponding
ITP for it, so I proceeded with my own packaging from scratch. I intend to maintain it
long-term and track upstream releases; this also unblocks packaging of
mangowc, which depends on it, but the package stands on its own merits.

I have prior contribution experience in Debian and have plenty of time
to dedicate to maintaining this package going forward.

Regards,
Damian Daniel

Control: retitle -1 RFS: scenefx/0.5-1 [ITP] -- wlroots scene API with eye-candy effects
Id like to note real quick: The *+ds-1 package is not made by me and is not what this RFS is for. Another person posted that package. I do not see a vision in their package as they have not filed an ITP or RFS. the exact package which this RFS is referring to is the first upload (0.5-1). I tried talking with the person responsible for th second upload but they didn't seem to wan't to cooperate or even discuss. I am hereby changing the title of this bug.

The dispute over maintainership should be solved first, then this can
go forward.
https://lists.debian.org/debian-mentors/2026/08/msg00113.html

I'm willing to sponsor this package under the condition you both work
together and this package is maintained in the Debian namespace of
salsa.

Hi Damian

- tinywl seems to be an embedded code library; nick's approach was to
  remove it (repacking the source package), that seems to be indicating
  that it is not used. Can you check if it is needed?
  If it is not used, you can also remove it using d/clean, so that it
  won't be used in the build - or you repack.
  If it is needed, it probably should be packaged separatly.

- d/copyright
  - "MIT" should be called "Expat"
  - The Comment for the MIT section is not needed, as we verbatimely
    document what upstream states in the source code.
  - There is an extra "not" in: "The full license text is not
    included here since it is __not__ part of
    /usr/share/common-licenses; see…"
    that negates the inteneded meaning.

    Said that, a better wording (as more standard) would be anyway
    something like:
    "On Debian systems, the full text of the Creative Commons Zero
    can be found in /usr/share/common-licenses/CC0-1.0"

- d/control:
  - don't specify version constraints on your BD, if they are fulfilled
    already since oldstable.
  - Stand-Version is usally only given with 3 digitt, not 4.
  - Rules-Requires-Root is obsolete, remove it.

- d/changelog should be a single line (Intial Release …), although I
  appreciate you've added Nick as uploader, you still should have only
  one entry in d/changelog.

- d/*.install
  use ${DEB_HOST_MULTIARCH} for the multiarch path, instead of the
  wildcard "*"

- should the examples be installed with dh_installexamples(1), maybe to
  the -dev package?

W: libscenefx-0.5: lacks-unversioned-link-to-shared-library example: usr/lib/x86_64-linux-gnu/libscenefx.so [usr/lib/x86_64-linux-gnu/libscenefx-0.5.so]
seems to indicate that there is something odd in regards to the SONAME
of th library. can you check?


Cheers,
--
tobi

Hi Tobi,
Thank you very much for the review, I've addressed everything you pointed out:

- tinywl: verified that it's not used (examples=false excludes it
already anyway), so re-packaged it as Files-Excluded in
debian/copyright.

- d/copyright: changed MIT to Expat, dropped the Comment in that
stanza, and dropped the CC0-1.0 stanza and text completely since
we don't ship tinywl anymore (so the double negative in wording
is irrelevant anymore, but I'll keep this in mind for the future).

- d/control: removed Rules-Requires-Root, and reduced the
Standards-Version to 4.7.4. Also, I went through each versioned
Build-Depends requirement and checked it directly against
bookworm (oldstable) -- none of them are actually satisfied
there (like libwayland-dev is 1.21.0 while mine is >= 1.24.0,
and meson is 1.0.1 while mine is >= 1.3), so I didn't change
those.

- d/changelog: squashed into a single entry.

- d/*.install: switched to ${DEB_HOST_MULTIARCH}.

- Examples: created debian/libscenefx-0.5-dev.examples so that
examples/scene-graph.c ships using dh_installexamples in the -dev
package.

- SONAME warning: it turns out that upstream's meson build doesn't
set a soversion, so there was only one output file
libscenefx-0.5.so and no unversioned symlink. Created .links file
so that -dev ships libscenefx.so

I also dropped the redundant Priority field and bumped debhelper-compat to 14. I pushed everything to the salsa repo and reuploaded onto debian mentors. Once again, thank you for the feedback. I hope to hear from you again!

Best regards,
Damian

Hi Damian,

thanks for the update.

As you are now repacking, you'll need to have a +ds suffix to the
upstream version, and update the watch file accordingly.

you can copy the watch file from Nick:
https://salsa.debian.org/nickh/scenefx/-/blob/debian/latest/debian/watch?ref_type=heads

Hi Tobi,
I added the +ds tag to the packages. I copied Nick's watch file as you suggested. I reimported upstream via uscan, rebuilt, re-uploaded to mentors, and pushed all changes to salsa. Thanks for pointing it out!

Let me know if anything else needs fixing.

Best Regards,
Damian

Uploaded, thanks for your contribution. It will now have to go through
NEW, but we can close this bug already.

Cheers,
tobi