Fixed FTBFS when building without tdehw but using HAL. #203

Merged
MicheleC merged 1 commits from fix/build-with-hal into master 3 years ago
Owner

This fixes build with HAL on and tdehw off. Tested on wheezy. Please help testing on BSD or slackware.

For debian testing, you can use this this tde-packaging commit to removed the unwanted files from .install files for both tdelibs and tdebase and to add missing dependencies for HAL.

This fixes build with HAL on and tdehw off. Tested on wheezy. Please help testing on BSD or slackware. For debian testing, you can use this [this tde-packaging](https://mirror.git.trinitydesktop.org/gitea/TDE/tde-packaging/commit/d40de40ddb445bc23a6907df38279c9e9d67d4db) commit to removed the unwanted files from .install files for both tdelibs and tdebase and to add missing dependencies for HAL.
MicheleC added this to the R14.1.0 release milestone 3 years ago
MicheleC force-pushed fix/build-with-hal from 1d39279e7b to fc3996fc30 3 years ago
Owner

Because HAL port on FreeBSD was removed, it complicates testing. I would have to restore a HAL port from some older copy and test whether HAL can be built on FreeBSD 13.

Because HAL port on FreeBSD was removed, it complicates testing. I would have to restore a HAL port from some older copy and test whether HAL can be built on FreeBSD 13.
Poster
Owner

Because HAL port on FreeBSD was removed, it complicates testing. I would have to restore a HAL port from some older copy and test whether HAL can be built on FreeBSD 13.

Tested in Wheezy was good. If test in Slackware from Greg is also good, I think we can skip testing in BSD, given than now HAL has been removed.

> Because HAL port on FreeBSD was removed, it complicates testing. I would have to restore a HAL port from some older copy and test whether HAL can be built on FreeBSD 13. Tested in Wheezy was good. If test in Slackware from Greg is also good, I think we can skip testing in BSD, given than now HAL has been removed.
Ghost commented 3 years ago

To my knowledge HAL is still available for NetBSD https://pkgsrc.se/sysutils/hal but no-one seems to have packaged TDE for this OS yet.

To my knowledge HAL is still available for NetBSD https://pkgsrc.se/sysutils/hal but no-one seems to have packaged TDE for this OS yet.
Poster
Owner

Isn't @obache building for NetBSD? Or is my memory wrong?

Isn't @obache building for NetBSD? Or is my memory wrong?
MicheleC requested review from SlavekB 3 years ago
MicheleC requested review from Ghost 3 years ago
Ghost commented 3 years ago

Isn't @obache building for NetBSD? Or is my memory wrong?

He might be building TDE through build scripts like Phillippe does on Arch.

> Isn't @obache building for NetBSD? Or is my memory wrong? He might be building TDE through build scripts like Phillippe does on Arch.
SlavekB approved these changes 3 years ago
SlavekB left a comment
Owner

It seems okay.

It seems okay.
Poster
Owner

It seems okay.

Thanks Slavek. I will wait till Greg's test this weekend before merging. Will be good to see a test on a non Debian distro, just for extra feedback.

>It seems okay. Thanks Slavek. I will wait till Greg's test this weekend before merging. Will be good to see a test on a non Debian distro, just for extra feedback.
MicheleC force-pushed fix/build-with-hal from fc3996fc30 to 9f7d035569 3 years ago
Poster
Owner

Rebased on latest master.

Rebased on latest master.
Ghost commented 3 years ago

Build fine with HAL only, CDs are detected as well. Good job! 👍

If you want me to perform other tests (HAL related), now is the time 😉

Build fine with HAL only, CDs are detected as well. Good job! :+1: If you want me to perform other tests (HAL related), now is the time :wink:
Ghost approved these changes 3 years ago
Poster
Owner

Great, thanks for testing Greg! I will need to understand why here .iso images are not detected correctly and if that is a VM specific issue (works fine in bullseye with tdehw though).

If you have a chance to create a LUKS disk and see if you can unlock/mount/unmount/lock it using HAL it would be great. That would be the last test.

Great, thanks for testing Greg! I will need to understand why here .iso images are not detected correctly and if that is a VM specific issue (works fine in bullseye with tdehw though). If you have a chance to create a LUKS disk and see if you can unlock/mount/unmount/lock it using HAL it would be great. That would be the last test.
MicheleC merged commit 9f7d035569 into master 3 years ago
MicheleC deleted branch fix/build-with-hal 3 years ago
Poster
Owner

If you have a chance to create a LUKS disk and see if you can unlock/mount/unmount/lock it using HAL it would be great. That would be the last test.

Hi Greg, I tested in Wheezy and there is definitely more work to do for LUKS support using HAL. It is doing some of the work but parts are still missing. No need to test for the time being, it would simply fail.

> If you have a chance to create a LUKS disk and see if you can unlock/mount/unmount/lock it using HAL it would be great. That would be the last test. Hi Greg, I tested in Wheezy and there is definitely more work to do for LUKS support using HAL. It is doing some of the work but parts are still missing. No need to test for the time being, it would simply fail.
Ghost commented 3 years ago

I'll keep this install (on an old hdd) somewhere in case further testing is needed.

Regarding lusk support with HAL, who's going to make use of It? Does Dillos relies on HAL?

I'll keep this install (on an old hdd) somewhere in case further testing is needed. Regarding lusk support with HAL, who's going to make use of It? Does Dillos relies on HAL?
Poster
Owner

Regarding lusk support with HAL, who's going to make use of It? Does Dillos relies on HAL?

No idea, but it is more a matter of doing things right. As long as we want to provide HAL support, we should try to make it as complete as possible.

> Regarding lusk support with HAL, who's going to make use of It? Does Dillos relies on HAL? > No idea, but it is more a matter of doing things right. As long as we want to provide HAL support, we should try to make it as complete as possible.

Reviewers

SlavekB approved these changes 3 years ago
The pull request has been merged as 9f7d035569.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: TDE/tdebase#203
Loading…
There is no content yet.