Re: speed-dreams-2.3.0: how to handle bundled jar files in spec file ?

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Am Di., 25. Juli 2023 um 10:19 Uhr schrieb Martin Gansser
<martin.gansser@xxxxxxxxx>:
>
> Hi,
>
> i want to package the current version of speed-dreams 2.3.0, but i noticed that jar files for the trackeditor are included in the package, but they already exist in the system.

Not really, there are two different things going on:

> compilation error:
> Checking for unpackaged file(s): /usr/lib/rpm/check-files /home/martin/rpmbuild/BUILDROOT/speed-dreams-2.3.0-1.fc38.x86_64
> error: Installed (but unpackaged) file(s) found:
>    /usr/bin/bsh-2.0b4.jar
>    /usr/bin/jdom-1.1.3.jar
>    /usr/bin/jgoodies-common-1.8.1.jar
>    /usr/bin/jgoodies-looks-2.5.3.jar

This just says that your spec file installs those files as part of the
build process (to the BUILDROOT, where check-files found them) but
that your spec file does not declare them (in the %files section).

> In the file src/tools/trackeditor/CMakeLists.txt from line 181 to line 230 the jar files are copied as you can see.
> https://sourceforge.net/p/speed-dreams/code/HEAD/tree/trunk/src/tools/trackeditor/CMakeLists.txt#l181

And that is why they get installed into the BUILDROOT.

I don't think /usr/bin is a good place for jars.

> do i have to use the jar files from the fedora packages now ?
>
> bsh-2.1.0-8.fc38.noarch
> jdom-1.1.3-32.fc38.noarch
> jgoodies-common-1.8.1-17.fc38.noarch
> jgoodies-looks-2.7.0-7.fc38.noarch

You should not "bundle" them with speed-dreams if they are available
as separate packages. In that sense: Yes.

BTW: The jdom package puts its jar into /usr/share/java/ as it should.

> that is, remove the jar files from the speed-dreams package and link to the Fedora jar files in the spec file.

speed-dreams should "require" those packages. And you will need to
adjust speed-dreams so that it uses them - by setting CLASSPATH if
necessary or whatever makes speed-dreams find those jars.

If your package comes with tests you might also need to buildrequire the others.

Cheers
Michael
_______________________________________________
devel mailing list -- devel@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/devel@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue




[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Fedora Announce]     [Fedora Users]     [Fedora Kernel]     [Fedora Testing]     [Fedora Formulas]     [Fedora PHP Devel]     [Kernel Development]     [Fedora Legacy]     [Fedora Maintainers]     [Fedora Desktop]     [PAM]     [Red Hat Development]     [Gimp]     [Yosemite News]

  Powered by Linux