mbruel / ngPost

Command Line (or minimalist GUI) usenet poster for binaries developped in C++/QT designed to be as fast as possible and offer all the main features to post data easily and safely. Releases for Linux, Windows and MacOS are available.
GNU General Public License v3.0
176 stars 33 forks source link

Code copied from newsup - Please give proper attribution #71

Closed demanuel closed 4 years ago

demanuel commented 4 years ago

Hi!

I just noticed that you copied (even the variable names are the same!) and adapted the yenc encoding code from my project to c++

my code: https://github.com/demanuel/NewsUP/blob/master/lib/NewsUP/yEnc.pm#L12

your code: https://github.com/mbruel/ngPost/blob/master/src/utils/Yenc.cpp#L24

As required from GPLv3 don't change the attribution!

This link in stackexchange contains a good explanation of the license. https://opensource.stackexchange.com/questions/4577/does-gplv3-require-attribution

How to give proper attribution: https://softwareengineering.stackexchange.com/questions/167935/proper-attribution-of-derived-work-in-a-gpl-project

Thanks!

Regards, David Santiago

mbruel commented 4 years ago

Hi David, yep, I took your yEnc and made it pure C++. I thought it was public domain as you don't have any header on your module and if I remember I found several other implementations that were similar. I thought I told you about it. Indeed I kept the variable names, cause the goal was not to appropriate the code to myself ;)

Are you happy with this header: //======================================================================== // // Copyright (C) 2019 Matthieu Bruel Matthieu.Bruel@gmail.com // Copyright (C) 2018 David Emanuel Santiago // Code derived from NewsUP project // Thank you Tomas Novysedlak for yenc encoding piece :-) // https://github.com/demanuel/NewsUP/commits/master/lib/NewsUP/yEnc.pm // // // This file is a part of ngPost : https://github.com/mbruel/ngPost // // ngPost is free software; you can redistribute it and/or modify // it under the terms of the GNU Lesser General Public License as // published by the Free Software Foundation; version 3.0 of the License. // // This program is distributed in the hope that it will be useful, // but WITHOUT ANY WARRANTY; without even the implied warranty of // MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the // GNU Lesser General Public License for more details. // You should have received a copy of the GNU Lesser General Public // License along with this program; if not, write to the Free Software // Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, // USA. // //========================================================================

I'd only add it in the cpp file.

On Fri, 14 Aug 2020 at 19:35, David Emanuel Santiago < notifications@github.com> wrote:

Hi!

I just noticed that you copied (even the variable names are the same!) and adapted the yenc encoding code from my project to c++

my code: https://github.com/demanuel/NewsUP/blob/master/lib/NewsUP/yEnc.pm#L12

your code: https://github.com/mbruel/ngPost/blob/master/src/utils/Yenc.cpp#L24

As required from GPLv3 don't change the attribution!

This link in stackexchange contains a good explanation of the license.

https://opensource.stackexchange.com/questions/4577/does-gplv3-require-attribution

How to give proper attribution:

https://softwareengineering.stackexchange.com/questions/167935/proper-attribution-of-derived-work-in-a-gpl-project

Thanks!

Regards, David Santiago

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/mbruel/ngPost/issues/71, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADSM773V42GX6L5GPZSWN63SAVYWNANCNFSM4P7WYUDA .

demanuel commented 4 years ago

Just this two lines are enough:

// Copyright (C) 2015-2018 David Emanuel Santiago and NewsUP contributors
// Code derived from NewsUP project

:-)

Thank you!

Regards, David Santiago

mbruel commented 4 years ago

cool it's done : https://github.com/mbruel/ngPost/blob/master/src/utils/Yenc.cpp sorry to forgot to do it in the first place... have you tried ngPost?

On Fri, 14 Aug 2020 at 23:18, David Emanuel Santiago < notifications@github.com> wrote:

Just this two lines are enough:

// Copyright (C) 2015-2018 David Emanuel Santiago and NewsUP contributors // Code derived from NewsUP project

:-)

Thank you!

Regards, David Santiago

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/mbruel/ngPost/issues/71#issuecomment-674276418, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADSM774ZEG55UFW7ARLZSGTSAWS4FANCNFSM4P7WYUDA .

demanuel commented 4 years ago

cool it's done Thank you :-)

have you tried ngPost? Yes i did. Once and it was fast :-)

Need to try it again :-)

Closing it.