Skip to content

Gajim 2.4.7#9380

Open
malfisya wants to merge 2 commits into
mainfrom
gajim
Open

Gajim 2.4.7#9380
malfisya wants to merge 2 commits into
mainfrom
gajim

Conversation

@malfisya

Copy link
Copy Markdown
Member

Summary

  • python-pysequoia: Add at v0.1.34
  • gajim: Update to v2.4.7

Test Plan

Launch Gajim

Checklist

  • Package was built and tested against unstable
  • This change could gainfully be listed in the weekly sync notes once merged
  • I agree to license this contribution and all my previous contributions under the licensing terms in LICENSE.md and have the power and authority to grant those licenses.

summary : OpenPGP in Python using Sequoia PGP
description: |
This library provides OpenPGP facilities in Python through the Sequoia PGP library.
networking : true

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What does it need networking for?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It pulls some rust crates during compile time.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you add a comment to the package.yml about that? It isn't obvious from just looking at the package :D

@EbonJaeger EbonJaeger left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Apparently I had to review it again; just commenting in the thread wasn't enough?

summary : OpenPGP in Python using Sequoia PGP
description: |
This library provides OpenPGP facilities in Python through the Sequoia PGP library.
networking : true

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you add a comment to the package.yml about that? It isn't obvious from just looking at the package :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Triage

Development

Successfully merging this pull request may close these issues.

2 participants