Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#9584 closed bug (fixed)

installoptionalpackage beoscompatibility gives error

Reported by: Kev Owned by: mmadia
Priority: normal Milestone: R1
Component: Build System Version: R1/alpha4.1
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: no Platform: x86

Description

/> installoptionalpackage beoscompatibility
To be installed:  BeOSCompatibility 
Warning: Adding BeOS compatibility symlinks. This will go away. Please fix your apps!
/boot/common/cache/tmp/install-optpkg.sh: line 286: AddDirectoryToHaikuImage: command not found
Generating a list of Package Names ...
... done.
/>

Did it actually install?

This is on hrev45413 gcc4hybrid.

Change History (11)

comment:1 Changed 6 years ago by anevilyak

Resolution: invalid
Status: newclosed

Yes. The warning is there to tell app devs that those links won't be available forever, since they're mainly there for badly written apps which hardcoded paths instead of using find_directory().

comment:2 Changed 6 years ago by Kev

Thanks, I understood the warning. What I wasn't sure about was:

AddDirectoryToHaikuImage: command not found

comment:3 Changed 6 years ago by anevilyak

Ah, didn't see that. Then no, it wasn't installed properly. However, installoptionalpackage is basically just a shell script hack that can only handle a subset of the full list of build-time optional packages, i.e. the ones that just unzip a file. More complex ones that use other jam directives can't be handled by it, such as this one and the Development packages. That's not realistically something that will be fixed since it will be superseded by the actual package management anyways.

Last edited 6 years ago by anevilyak (previous) (diff)

comment:4 Changed 6 years ago by umccullough

At the very least, Matt could blacklist this package from the script if he doesn't have motivation to support the AddDirectoryToHaikuImage rule.

comment:5 Changed 6 years ago by mmadia

Resolution: invalid
Status: closedreopened

comment:6 Changed 6 years ago by mmadia

Owner: changed from bonefish to mmadia
Status: reopenedin-progress

It'll be easy enough to implement AddDirectoryToHaikuImage in installoptionalpackage.

comment:7 Changed 6 years ago by mmadia

Resolution: fixed
Status: in-progressclosed

Implemented in hrev45414. Thanks!

comment:8 Changed 6 years ago by Kev

There are some possibly abandoned titles like Bracker that it would be great to see if they still worked. Thanks for your help! -K

comment:9 in reply to:  8 Changed 6 years ago by anevilyak

Replying to Kev:

There are some possibly abandoned titles like Bracker that it would be great to see if they still worked. Thanks for your help! -K

Just to be clear: this package isn't a requirement to run R5 apps in general, it only exists to support a handful of apps that aren't well-behaved and make assumptions about the on-disk folder layout.

comment:10 Changed 6 years ago by Kev

So the kind of thing that would fix a 'missing lib' error but not a crash.

comment:11 in reply to:  10 Changed 6 years ago by anevilyak

Replying to Kev:

So the kind of thing that would fix a 'missing lib' error but not a crash.

Mm, no, missing lib would be a different issue. What it'd be most likely to fix is random errors/complaints about not being able to find/read/write various files while the app's running.

Last edited 6 years ago by anevilyak (previous) (diff)
Note: See TracTickets for help on using tickets.