Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add good old Beastie #25

Merged
merged 1 commit into from
Oct 4, 2022
Merged

Add good old Beastie #25

merged 1 commit into from
Oct 4, 2022

Conversation

ehaupt
Copy link
Contributor

@ehaupt ehaupt commented Oct 4, 2022

Now that there is an official FreeBSD port we definitely need some proper representation 😃

Now that there is an official [FreeBSD port](freebsd/freebsd-ports@7f2f0cc) we definitely need some proper representation 😃
@poetaman
Copy link
Owner

poetaman commented Oct 4, 2022

@ehaupt Excellent! Thanks for adding arttime to official FreeBSD ports. Yes it would be good to have Beastie now that we have FreeBSD support :). A few points about port:

  • My repository's username needs update in your port.
  • Games is not the most accurate description, even though it can be used as a random art selection game. I use it mostly as a timer to get desktop notifications while multi-tasking or time-managing. May I propose one new directory like: devtools, productivity, time, multitasking? There are so many applications available on GitHub that are developer productivity focused, at least productivity deserves a directory.
  • How could this be made into a package? So user can just do pkg install arttime if they don't want to install the ports system? On my system ports fetch extract populated the /usr/ports directory with 800MB of content. I had previously tried that and removed the pkg system as my use case was disk space constrained. Could you help make pkg install arttime work?

Also, I have fixed the spacing to make Beastie center on the screen. Check the comment 3645667#commitcomment-85854953. Unfortunately given the port points to arttime 1.9.1, Beastie won't show in ports till next release...

@poetaman poetaman merged commit 2cd785d into poetaman:main Oct 4, 2022
@ehaupt
Copy link
Contributor Author

ehaupt commented Oct 6, 2022

@ehaupt Excellent! Thanks for adding arttime to official FreeBSD ports. Yes it would be good to have Beastie now that we have FreeBSD support :). A few points about port:

* My repository's username needs update in your port.

Seems like you've renamed your account.

https://github.com/reportaman/arttime

redirects to:

https://github.com/poetaman/arttime

I'll update it accordingly.

* Games is not the most accurate description, even though it can be used as a random art selection game. I use it mostly as a timer to get desktop notifications while multi-tasking or time-managing. May I propose one new directory like: devtools, productivity, time, multitasking? There are so many applications available on GitHub that are developer productivity focused, at least productivity deserves a directory.

That is not official ports category that exists.

* How could this be made into a package? So user can just do `pkg install arttime` if they don't want to install the ports system? On my system `ports fetch extract` populated the `/usr/ports` directory with 800MB of content. I had previously tried that and removed the pkg system as my use case was disk space constrained. Could you help make `pkg install arttime` work?

I recommend reading the following documentation:

https://docs.freebsd.org/en/books/handbook/ports/

This command will work once the official package builders have caught on:

pkg install arttime

That should be the case for the latest package branch within roughly one week.

Also, I have fixed the spacing to make Beastie center on the screen. Check the comment 3645667#commitcomment-85854953. Unfortunately given the port points to arttime 1.9.1, Beastie won't show in ports till next release...

Awesome, thanks.

@poetaman
Copy link
Owner

poetaman commented Oct 6, 2022

@ehaupt Ah thanks for the info! Dependency for native desktop sound would change soon based on @grahamperrin's suggestions in #26 (comment), and our findings in discussion #29 and ticket #27. Will keep you posted!

As a side note, you too got a shout-out in the release notes of v1.9.1 release notes. Thanks so much for the support, and explanations!

@ehaupt ehaupt deleted the patch-1 branch October 6, 2022 08:51
@ehaupt
Copy link
Contributor Author

ehaupt commented Oct 6, 2022

@ehaupt Ah thanks for the info! Dependency for native desktop sound would change soon based on @grahamperrin's suggestions in #26 (comment), and our findings in #29. Will keep you posted!

Thanks, I'll keep this in mind once a new version gets released (adding such changes to release notes helps).

As a side note, you too got a shout-out in the release notes of v1.9.1 release notes. Thanks so much for the support, and explanations!

It's my pleasure. Thank you for the shout-out.

May I suggest adding a repology badge into your README.md? It adds awareness of the packaging status of your project. It's also a good overview for version distribution across different package repositories, plus it leads users to repo specific information on how to install the package. Many projects add those badges (eg.: https://github.com/bloomreach/s4cmd).

@poetaman
Copy link
Owner

poetaman commented Oct 6, 2022

@ehaupt Great suggestion! Would you like to create a PR for Readme.md? I can merge it tomorrow, it's a little late here. If not then I'll figure it tomorrow. Thanks :)

ehaupt added a commit to ehaupt/arttime that referenced this pull request Oct 6, 2022
As discussed in poetaman#25, add repology badge.
@ehaupt ehaupt mentioned this pull request Oct 6, 2022
poetaman added a commit that referenced this pull request Dec 5, 2022
arttime made it to official FreeBSD ports, adding good old FreeBSD Beastie. Cheers!
poetaman pushed a commit that referenced this pull request Dec 5, 2022
As discussed in #25, add repology badge.
poetaman added a commit that referenced this pull request Dec 4, 2023
arttime made it to official FreeBSD ports, adding good old FreeBSD Beastie. Cheers!
poetaman pushed a commit that referenced this pull request Dec 4, 2023
As discussed in #25, add repology badge.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants