WIP: Add `TDEMultimedia` ebuilds. #157

Closed
Chris wants to merge 1 commits from feat/tdemultimedia into master
Chris commented 2 years ago
Collaborator

Noatun isn't building, because it seems to need MPEGLIB (while building!), which needs aRts (while building!) and aRts needs some aRts plugins (while building!) and further and some things are unclear.

JuK, seems to need aKode to build, but at the same time, it can output to aRts and use aRts for decoding, which would allow to make aKode optional, for example. But that isn't the case. In fact, it seems that was possible in KDE3.

KSCD fails to build, even if kscddb is installed already. All aRts related stuff seems to depend on each other (while building!), which makes building them seperate difficult. Some things are missing. So it is still WIP.

It seems here is work needed at TDE level related to the CMake files to make it possible to build components seperately and link against the installed versions. But it can't be wrong already to work on the ebuilds.

Reference: https://mirror.git.trinitydesktop.org/gitea/TDE/tdemultimedia/src/branch/master/CMakeLists.txt#L112

Components which can't be seperately build:

  • MPEGLIB (Seems to need aRts)
  • Noatun (Seems to need MPEGLIB, but even if that is installed, it can't find it)
  • KSCD (Seems to need libkcddb)
  • KREC (Seems to need KSCD)
  • KaudioCreator (Seems to need KSCD)
  • TDEIOSlaves (Seems to need KSCD)

Also it seems JuK had some -WITH_ARTS back in the days, which isn't existing anymore in the CMake conversion.

Signed-off-by: Chris xchrisx@uber.space

Noatun isn't building, because it seems to need MPEGLIB (while building!), which needs aRts (while building!) and aRts needs some aRts plugins (while building!) and further and some things are unclear. JuK, seems to need aKode to build, but at the same time, it can output to aRts and use aRts for decoding, which would allow to make aKode optional, for example. But that isn't the case. In fact, it seems that was possible in KDE3. KSCD fails to build, even if kscddb is installed already. All aRts related stuff seems to depend on each other (while building!), which makes building them seperate difficult. Some things are missing. So it is still WIP. It seems here is work needed at TDE level related to the CMake files to make it possible to build components seperately and link against the installed versions. But it can't be wrong already to work on the ebuilds. **Reference:** https://mirror.git.trinitydesktop.org/gitea/TDE/tdemultimedia/src/branch/master/CMakeLists.txt#L112 **Components which can't be seperately build:** - MPEGLIB (Seems to need aRts) - Noatun (Seems to need MPEGLIB, but even if that is installed, it can't find it) - KSCD (Seems to need libkcddb) - KREC (Seems to need KSCD) - KaudioCreator (Seems to need KSCD) - TDEIOSlaves (Seems to need KSCD) Also it seems `JuK` had some `-WITH_ARTS` back in the days, which isn't existing anymore in the CMake conversion. Signed-off-by: Chris <xchrisx@uber.space>
Chris added this to the R14.1.0 release milestone 2 years ago
Chris added the
PR/wip
label 2 years ago
SlavekB force-pushed feat/tdemultimedia from 786463ab22 to 68abe0f485 3 months ago
Owner

I did rebase and resolve conflicts of this old PR and seem to be the only difference that is here is a separate ebuild for mpeglib. If I understand it correctly, mpeglib is now built within ebuild tdemultimedia-arts, so separate ebuild for mpeglib is not necessary?

I did rebase and resolve conflicts of this old PR and seem to be the only difference that is here is a separate ebuild for mpeglib. If I understand it correctly, mpeglib is now built within ebuild tdemultimedia-arts, so separate ebuild for mpeglib is not necessary?
ormorph force-pushed feat/tdemultimedia from 68abe0f485 to 833e8a8458 2 months ago
ormorph changed title from WIP: Live ebuilds: Add `TDEMultimedia` ebuilds. to Live ebuilds: Add `TDEMultimedia` ebuilds. 2 months ago
ormorph changed title from Live ebuilds: Add `TDEMultimedia` ebuilds. to Bug fixes #208 and #207 2 months ago
Collaborator

I reloaded it into this thread, I don't know how it happened))
Since these changes are out of date, I changed the title. You can accept a new commit.

I reloaded it into this thread, I don't know how it happened)) Since these changes are out of date, I changed the title. You can accept a new commit.
Collaborator

Hmm, I found the previous changes for this branch. Now I will return everything for history and re-upload a new branch, and create a new PR.

Hmm, I found the previous changes for this branch. Now I will return everything for history and re-upload a new branch, and create a new PR.
ormorph force-pushed feat/tdemultimedia from 833e8a8458 to 786463ab22 2 months ago
ormorph changed title from Bug fixes #208 and #207 to Wip: TDEMultimedia` ebuilds. на Live ebuilds: Add `TDEMultimedia` ebuilds. 2 months ago
ormorph changed title from Wip: TDEMultimedia` ebuilds. на Live ebuilds: Add `TDEMultimedia` ebuilds. to WIP: TDEMultimedia` ebuilds. на Live ebuilds: Add `TDEMultimedia` ebuilds. 2 months ago
SlavekB force-pushed feat/tdemultimedia from 786463ab22 to 4799e0c00b 2 months ago
Owner

The previous override occurred due to the fact that you chose the same too general name of branch as is used in this PR. I now pushed the change as I recently made rebase and merge with current state – see my previous comment.

The previous override occurred due to the fact that you chose the same too general name of branch as is used in this PR. I now pushed the change as I recently made rebase and merge with current state – see [my previous comment](pulls/157#issuecomment-16068).
Collaborator

Hmm, basically I can make mpeglib separate from tdemultimedia-arts. Mpeglib is currently part of tdemultimedia-arts. If you plan to use mpeglib separately, without using arts, then this can be implemented.

Hmm, basically I can make mpeglib separate from tdemultimedia-arts. Mpeglib is currently part of tdemultimedia-arts. If you plan to use mpeglib separately, without using arts, then this can be implemented.
Owner

Hmm, basically I can make mpeglib separate from tdemultimedia-arts. Mpeglib is currently part of tdemultimedia-arts. If you plan to use mpeglib separately, without using arts, then this can be implemented.

I watched mpeglib is dependent on aRts, so separate use without aRts is not possible. So it seems to make sense to be part of tdemultimedia-arts.

Because all other parts of this PR were already resolved within other PR, this PR probably may be closed without merging a change on mpeglib. What's your opinion?

> Hmm, basically I can make mpeglib separate from tdemultimedia-arts. Mpeglib is currently part of tdemultimedia-arts. If you plan to use mpeglib separately, without using arts, then this can be implemented. > I watched mpeglib is dependent on aRts, so separate use without aRts is not possible. So it seems to make sense to be part of tdemultimedia-arts. Because all other parts of this PR were already resolved within other PR, this PR probably may be closed without merging a change on mpeglib. What's your opinion?
SlavekB changed title from WIP: TDEMultimedia` ebuilds. на Live ebuilds: Add `TDEMultimedia` ebuilds. to WIP: Add `TDEMultimedia` ebuilds. 2 months ago
Collaborator

I watched mpeglib is dependent on aRts, so separate use without aRts is not possible. So it seems to make sense to be part of tdemultimedia-arts.

Checked it. I agree this PR can be closed. At least right now it is impossible to build mpeglib without arts.

> I watched mpeglib is dependent on aRts, so separate use without aRts is not possible. So it seems to make sense to be part of tdemultimedia-arts. Checked it. I agree this PR can be closed. At least right now it is impossible to build mpeglib without arts.
SlavekB removed this from the R14.1.0 release milestone 2 months ago
SlavekB removed the
PR/wip
label 2 months ago
SlavekB closed this pull request 2 months ago
SlavekB deleted branch feat/tdemultimedia 2 months ago
This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.