Skip to content

Clarification on backport of pyelliptic requested + external dependency unbundling #886

Open

Description

As I work for a GNU system which unbundles where possible, I'd like to get some more
clarification on the backporting of pyelliptic you do. Can you explain your changes or point
out in the code which changes are made?

At your choice, could you even move the external dependencies
into git-submodules (other repositories) so it can be up to package
maintainers to pull them in?

This would improve the work of us package maintainers in cutting down the lines of code, especially for package systems like GNU Guix or systems like Gentoo.
https://wiki.gentoo.org/wiki/Why_not_bundle_dependencies
http://dustycloud.org/misc/talks/guix/chicagolug_2015/guix_talk.html (could include reasons and understanding, there are very likely more insights in https://www.gnu.org/software/guix/help/#papers and https://www.gnu.org/software/guix/help/#talks (same page))

Activity

  1. PeterSurda commented on Jul 16, 2016

    @PeterSurda
    Member

    I don't know what the difference is, you'd have to diff it yourself. You could contact Atheros and ask him.

  2. self-assigned this
    on Jul 16, 2016
  3. bmng-dev commented on Jul 17, 2016

    @bmng-dev
    Contributor

    Newer versions of PyElliptic break the Bitmessage protocol. Bitmessage still uses ECDSA-with-SHA1 signatures and length prefixed coordinate representation for ephemeral public keys while recent versions of PyElliptic only use ECDSA-with-SHA2 signatures and uncompressed point compression form representation for ephemeral public keys. The bundled PyElliptic has an improved OpenSSL library locator.

    The bundled Socksipy has been modified to support Tor's SOCKS Extensions.

  4. rekado commented on Jul 17, 2016

    @rekado

    Do you plan to send your improvements to the respective upstream projects?

  5. bmng-dev commented on Jul 19, 2016

    @bmng-dev
    Contributor

    Me personally? No, I didn't make the modifications.

  6. PeterSurda commented on Jul 19, 2016

    @PeterSurda
    Member

    I modified the socksipy, I added support for the RESOLVE extension used by Tor. This is used for improving bootstrapping. I can try to push it upstream.

    PyElliptic was probably modified by Atheros (I wasn't involved in the development at that time so I don't know).

  7. added this to the v0.6.2 milestone on Feb 22, 2017
  8. PeterSurda commented on Feb 26, 2017

    @PeterSurda
    Member

    Also recently me and one other developer modified the bundled pyelliptic to support OpenSSL 1.1.x and LibreSSL (the old code does support LibreSSL, but if you want to add OpenSSL 1.1.x support you also have to distinguish between OpenSSL and LibreSSL). I'll see if it can be pushed upstream, but it's not a high priority and the upstream seems abandoned.

    I'll refactor socksify into an asyncore class in the future, so the socksipy dependecy can be removed altogether.

  9. modified the milestones: v0.6.3, v0.6.2 on Feb 26, 2017
  10. g1itch commented on Mar 2, 2017

    @g1itch
    Collaborator

    Concerning socksocket.resolve() there is an issue in the upstream PySocks project. I think you can cooperate.

  11. added a commit that references this issue on Jul 3, 2018
  12. modified the milestones: v0.6.3, v0.6.4 on Aug 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions