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

Update the DeforaOS Desktop packages to the latest versions available #309

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

khorben
Copy link
Contributor

@khorben khorben commented Oct 11, 2024

This updates a number of packages relevant to the DeforaOS Desktop environment, and imports one as well:

  • Editor 0.4.1
  • Integration 0.1.3
  • Keyboard 0.4.0
  • libDesktop 0.4.2
  • Mailer 0.2.0
  • Notes 0.2.1 (new import)
  • Panel 0.4.6
  • Player 0.2.0
  • Todo 0.3.1

Changes suggested by portclippy(1), portfmt(1), and portlint(1) have been applied on every package.

Tested on FreeBSD/amd64 with Xorg.

Additional fixes for Wayland support are still pending changes upstream, together with the corresponding releases.

@khorben
Copy link
Contributor Author

khorben commented Oct 11, 2024

Cc @Jehops, @ocochard; thanks!

@ocochard
Copy link
Member

ok for portlint, but did you test them with a poudriere testport ?

poudriere testport -j builder -p deforaos mail/deforaos-mailer
(etc.)
===> Checking for items in STAGEDIR missing from pkg-plist
Error: Orphaned: share/doc/Mailer/COPYING
===> Checking for items in pkg-plist which are not in STAGEDIR
===> Error: Plist issues found.

About editors/deforaos-editor:

===> Checking for items in STAGEDIR missing from pkg-plist
Error: Orphaned: share/man/html1/editor.html.gz
Error: Orphaned: share/man/html1/filter.html.gz
===> Checking for items in pkg-plist which are not in STAGEDIR
===> Error: Plist issues found.

@Jehops
Copy link
Member

Jehops commented Oct 11, 2024

@khorben, these ports also have plist issues:

  • editors/deforaos-editor
  • deskutils/deforaos-notes
  • deskutils/deforaos-todo

@khorben
Copy link
Contributor Author

khorben commented Oct 11, 2024

ok for portlint, but did you test them with a poudriere testport ? [...]

No, but thanks for the heads up!

I can set it up on my side, unless one of you beats me to it 😁

@khorben khorben force-pushed the khorben/deforaos-updates-20241010 branch from 7f9e2e2 to efef504 Compare October 12, 2024 17:20
@khorben
Copy link
Contributor Author

khorben commented Oct 12, 2024

ok for portlint, but did you test them with a poudriere testport ? [...]

No, but thanks for the heads up!

I can set it up on my side, unless one of you beats me to it 😁

I have setup an instance of poudriere, and found and fixed some issues already. I haven't tested everything completely yet.

@khorben khorben force-pushed the khorben/deforaos-updates-20241010 branch 2 times, most recently from 5cb02be to e73c2f9 Compare October 12, 2024 22:35
@khorben
Copy link
Contributor Author

khorben commented Oct 12, 2024

I have setup an instance of poudriere, and found and fixed some issues already. I haven't tested everything completely yet.

I have fixed a couple more issues in editors/deforaos-editor and mail/deforaos-mailer.

@Jehops
Copy link
Member

Jehops commented Oct 13, 2024

For the two remaining plist errors, you need to add share/doc/Todo/COPYING to deskutils/deforaos-todo/pkg-plist and share/doc/Notes/COPYING to deskutils/deforaos-notes/pkg-plist.

@khorben khorben force-pushed the khorben/deforaos-updates-20241010 branch from e73c2f9 to 8b4cea4 Compare October 13, 2024 10:09
@khorben
Copy link
Contributor Author

khorben commented Oct 13, 2024

Thanks, I have done that, and added an NLS option to www/deforaos-surfer as well.

Sponsored by:	The FreeBSD Foundation
Sponsored by:	The FreeBSD Foundation
Sponsored by:	The FreeBSD Foundation
Sponsored by:	The FreeBSD Foundation
Sponsored by:	The FreeBSD Foundation
@khorben khorben force-pushed the khorben/deforaos-updates-20241010 branch from 8b4cea4 to 3adcd74 Compare October 14, 2024 15:30
Sponsored by:	The FreeBSD Foundation
Sponsored by:	The FreeBSD Foundation
Sponsored by:	The FreeBSD Foundation
Sponsored by:	The FreeBSD Foundation
This lets the deforaos-surfer port pass the plist, portclippy, portfmt,
and portlint tests. This notably adds an "NLS" option, and bumps the
PORTREVISION as a consequence as well.

Sponsored by:	The FreeBSD Foundation
@khorben khorben force-pushed the khorben/deforaos-updates-20241010 branch from 3adcd74 to 26b5061 Compare October 17, 2024 17:45
@khorben
Copy link
Contributor Author

khorben commented Oct 17, 2024

I have fixed another issue, where a dependency was missing for mail/deforaos-mailer, namely on textproc/gtk-doc for the DOCS option.

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.

3 participants