Skip to content

module!: author should take a list #244

Closed

Description

@ojeda

MODULE_AUTHOR is supposed to be repeated in C.

Perhaps we should rename it authors, too.

Activity

  1. wcampbell0x2a commented on May 6, 2021

    @wcampbell0x2a

    Is this the API you are looking for?

    -    author: b"Rust for Linux Contributors",
    +    author: [b"Rust for Linux Contributors", b"john@doe"],
    
  2. ojeda commented on May 6, 2021

    @ojeda
    MemberAuthor

    Something like that, yeah!

  3. GuilhermeGiacomoSimoes commented on Dec 6, 2024

    @GuilhermeGiacomoSimoes
  4. ojeda commented on Dec 6, 2024

    @ojeda
    MemberAuthor

    Thanks!

    @wcampbell0x2a submitted a PR a while ago (#904) that was reviewed etc., did you see that? I think the patches were never submitted, though.

  5. GuilhermeGiacomoSimoes commented on Dec 6, 2024

    @GuilhermeGiacomoSimoes

    @ojeda

    Yes, I saw that, but since @wcampbell0x2a never sent his patches, and it was a long time ago, I did the change my way and sent my patches.

  6. ojeda commented on Dec 6, 2024

    @ojeda
    MemberAuthor

    In general, I recommend contacting the person, even if just out of deference. If there is no reply, then you could still consider submitting their patches etc.

    Moreover, it is also a good idea to mention prior work in the commit message, and discuss why you used a different approach, since the PR was discussed by a few people back then.

    Thanks!

  7. 4 remaining items

  8. added a commit that references this issue on Mar 10, 2025
  9. ojeda commented on Mar 10, 2025

    @ojeda
    MemberAuthor

    https://lore.kernel.org/rust-for-linux/20250309175712.845622-1-trintaeoitogc@gmail.com/

    Applied to rust-next, so closing since it will likely land in mainline.

  10. added a commit that references this issue on Mar 10, 2025
  11. added a commit that references this issue on Apr 11, 2025
  12. added a commit that references this issue on May 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    • libRelated to the `rust/` library.

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions