serenity icon indicating copy to clipboard operation
serenity copied to clipboard

Ports: Add cbonsai port

Open tarob0ba opened this issue 2 years ago • 5 comments

This PR modifies the port list and adds the package.sh script for cbonsai, a delightful program that generates bonsai trees in your terminal.

tarob0ba avatar Jun 24 '23 04:06 tarob0ba

Hello!

One or more of the commit messages in this PR do not match the SerenityOS code submission policy, please check the lint_commits CI job for more details on which commits were flagged and why. Please do not close this PR and open another, instead modify your commit message(s) with git commit --amend and force push those changes to update this PR.

BuggieBot avatar Jun 24 '23 04:06 BuggieBot

Fixing the table row, sorry. Next time, I'll remember to run the lint script.

Edit: fixed.

tarob0ba avatar Jun 24 '23 04:06 tarob0ba

The package.sh script isn't marked as executable. I think you need to chmod +x it.

AtkinsSJ avatar Jun 24 '23 10:06 AtkinsSJ

After trying it out, I get crashes in strtold() when trying to provide any of the command-line arguments. :grimacing: It fails assert(sizeof(double) == sizeof(long double));

(This is a problem with libc, not with your PR, but it is annoying...)

AtkinsSJ avatar Jun 24 '23 10:06 AtkinsSJ

Huh. The file was marked as executable, but Git marked it as 644 anyways. Just ran git update-index --chmod=+x and now it has registered.

tarob0ba avatar Jun 24 '23 14:06 tarob0ba

Looks like it still managed to not be executable. (https://github.com/SerenityOS/serenity/pull/19622) :thonk: Maybe something about your system is messing with it. Are you on Windows?

AtkinsSJ avatar Jun 25 '23 11:06 AtkinsSJ

Nope, I’m running Ubuntu. I ran git ls-files -s package.sh before updating the commit and it displayed 744 in the file perms. I have no idea what’s going on.

tarob0ba avatar Jun 25 '23 12:06 tarob0ba