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 entries for Beej's Guides #6781

Merged
merged 19 commits into from
Mar 18, 2022
Merged

Update entries for Beej's Guides #6781

merged 19 commits into from
Mar 18, 2022

Conversation

jayaddison
Copy link
Contributor

@jayaddison jayaddison commented Mar 16, 2022

What does this PR do?

For Beej's Guide to Networking (canonical link):

  • Adds 1x reference within the books-by-subject (section: Networking)
  • Updates 1x reference with file format availability in books-by-programming-language (section: C)
  • Adds 4x references to translations of the book per-natural-language (sections: Dansk, Magyar, Române, Srpski)

From code review feedback:

  • Updates additional and existing references to author 'B. Hall' to use full name 'Brian "Beej Jorgensen" Hall'

For resources

Description

Why is this valuable (or not)?

"Beej's Guide to Networking" is a valuable guide to network programming in C.

How do we know it's really free?

The work is currently "Attribution-NonCommercial-NoDerivs 3.0" licensed, as indicated in the publication's source: https://github.com/beejjorgensen/bgnet/blob/cce225ad6856b749bf92a81ed26208785ed5adb9/src/bgnet.md#copyright-distribution-and-legal-legal

One exception to this is that the author requires:

"One specific exception to the "No Derivative Works" portion of the license is as follows: this guide may be freely translated into any language, provided the translation is accurate, and the guide is reprinted in its entirety. The same license restrictions apply to the translation as to the original guide. The translation may also include the name and contact information for the translator."

All of the translations linked to here are listed on the author's own website for the book.

For book lists, is it a book? For course lists, is it a course? etc.

This is a book; it's canonically available in English on the web in HTML and PDF formats, and also has published paper formats, and a number of online translations available.

Checklist:

  • Read our contributing guidelines
  • Search for duplicates.
  • Include author(s) and platform where appropriate.
  • Put lists in alphabetical order, correct spacing.
  • Add needed indications (PDF, access notes, under construction)

Follow-up

  • Check the status of GitHub Actions and resolve any reported warnings!

cc @beejjorgensen (author)

Copy link
Member

@davorpa davorpa left a comment

Choose a reason for hiding this comment

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

Moreover, when we prefer the full name of author: Brian "Beej Jorgensen" Hall

we should provide the name of the traducer in translation entries too when would be posible

@davorpa davorpa added 🗣️ locale:en Resources addressing "English" language 🗣️ locale:da Resources addressing "Danish / dansk" language 🗣️ locale:hu Resources addressing "Hungarian / magyar / magyar nyelv" language 🗣️ locale:ro Resources addressing "Romanian (Romania) / limba română / român" language waiting for changes PR has been reviewed and changes/suggestions requested 🗣️ locale:sr Resources addressing "Serbian / српски језик / srpski jezik" language labels Mar 17, 2022
@jayaddison
Copy link
Contributor Author

Thanks @davorpa for the review!

Moreover, when we prefer the full name of author: Brian "Beej Jorgensen" Hall

Could/should guidance around that be added in the CONTRIBUTING.md? (I'm preparing those changes, although would like to confirm and have it documented for future contributors too)

we should provide the name of the traducer in translation entries too when would be posible

Yep, agreed (providing credit where it is due). It was a little lazy of me not to go through and attempt to collect those credits while following these links. I'll take another read through for them soon.

@jayaddison jayaddison changed the title Update entries for Beej's Guide to Network Programming Update entries for Beej's Guides Mar 17, 2022
@jayaddison
Copy link
Contributor Author

jayaddison commented Mar 17, 2022

Please note: there is an additional translation in Farsi that is not yet added here. I'm unable to read that language at the moment and wasn't able to determine what the title of the translated book is/was.

Copy link
Collaborator

@eshellman eshellman left a comment

Choose a reason for hiding this comment

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

Wow this is some great work!

books/free-programming-books-subjects.md Outdated Show resolved Hide resolved
@davorpa davorpa added 🗣️ locale:de Resources addressing "German / Deutsch" language 🗣️ locale:fr Resources addressing "French / français" language 🗣️ locale:it Resources addressing "Italian / italiano" language 🗣️ locale:pl Resources addressing "Polish / polski" language 🗣️ locale:ru Resources addressing "Russian / Русский язык" language 🗣️ locale:zh Resources addressing "Chinese" language 🗣️ locale:tr Resources addressing "Turkish / Türkçe" language labels Mar 17, 2022
books/free-programming-books-it.md Outdated Show resolved Hide resolved
jayaddison and others added 3 commits March 17, 2022 17:13
… name

Co-authored-by: David Ordás <3125580+davorpa@users.noreply.github.com>
…anguage (guideline: one entry per book per human-language; 'Networking' is approved as the better location)
@jayaddison
Copy link
Contributor Author

Thanks @eshellman!

Copy link
Member

@davorpa davorpa left a comment

Choose a reason for hiding this comment

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

LGTM

@davorpa davorpa added 🚀 ready to merge LGTM. Waiting for final approval and removed waiting for changes PR has been reviewed and changes/suggestions requested labels Mar 17, 2022
@eshellman
Copy link
Collaborator

Thanks, and welcome to free-programming-books!

@eshellman eshellman merged commit de3a320 into EbookFoundation:main Mar 18, 2022
@jayaddison jayaddison deleted the maintenance/sync-beejs-guide-to-networking branch March 18, 2022 09:43
@davorpa davorpa mentioned this pull request Mar 18, 2022
39 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚀 ready to merge LGTM. Waiting for final approval 🗣️ locale:da Resources addressing "Danish / dansk" language 🗣️ locale:de Resources addressing "German / Deutsch" language 🗣️ locale:en Resources addressing "English" language 🗣️ locale:fr Resources addressing "French / français" language 🗣️ locale:hu Resources addressing "Hungarian / magyar / magyar nyelv" language 🗣️ locale:it Resources addressing "Italian / italiano" language 🗣️ locale:pl Resources addressing "Polish / polski" language 🗣️ locale:ro Resources addressing "Romanian (Romania) / limba română / român" language 🗣️ locale:ru Resources addressing "Russian / Русский язык" language 🗣️ locale:sr Resources addressing "Serbian / српски језик / srpski jezik" language 🗣️ locale:tr Resources addressing "Turkish / Türkçe" language 🗣️ locale:zh Resources addressing "Chinese" language
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants