Hello guys,

Sometimes when I look at the BOINC repo on GitHub and see many open pull
requests (22 at this time) which are opened many days or weeks or months
this makes me sad. And there are no any activities with these pull
requests. Some pull requests have no answers and comments from the
committee. It looks like no one are interested in the help of third-party
contributors. And this makes me sad more.
I want to change this. I want this because I think that this application is
very important for our world. That is why I want to see it developing.
Unfortunately I can do almost nothing in this situation so I made a
decision to write monthly digest with list of opened pull requests and my
comments about them. Maybe this mail will let you to spend a couple of
minutes of your time to make some decision about these pull requests.
So, let's start.

*July 2017, 22 open pull requests.*

#1604: BOINC GAHP: print SVN_VERSION with --version CLA
Link: https://github.com/BOINC/boinc/pull/1604
Opened: 4 Aug 2016
*Age: 360 days*
Status: 1 comment from Christian Beer. No other actions.
Comments: GAHP specific. Some resolution needed either to accept
Christian's proposition or decline this pull request completely.

#1605: BOINC GAHP condor input template
Link: https://github.com/BOINC/boinc/pull/1605
Opened: 4 Aug 2016
*Age: 360 days*
Status: *No comments*
Comments: GAHP specific. Should be reviewed.

#1638: Fix c++11 build
Link: https://github.com/BOINC/boinc/pull/1638
Opened: 2 Sep 2016
*Age: 331 days*
Status: Work suspended.
Comments: Should be reviewed by someone who is familiar with FastCGI.

#1821: Build: restructure build process on Mac to make it more flexible and
allow CI support
Link: https://github.com/BOINC/boinc/pull/1821
Opened: 17 Mar 2017
*Age: 135 days*
Status: Work seems to be finished. Waiting for review. *No comments.*
Comments: This is very helpfull pull request. Should be added to CI as from
my POV.

#1827: Remote job submission: first draft of protocol spec file for
BOINC-GAHP
Link: https://github.com/BOINC/boinc/pull/1827
Opened: 24 Mar 2017
*Age: 128 days*
Status: Need to be reviewed.
Comments: GAHP related documentation.

#1854: Manager: display zero free space in disk tab
Link: https://github.com/BOINC/boinc/pull/1854
Opened: 6 Apr 2017
*Age: 115 days*
Status: Ready for review. *No comments.*
Comments: *This is a fix for an issue which was originally found 9(!) years
ago.*

#1855: Build: install locale files in out-of-tree builds
Link: https://github.com/BOINC/boinc/pull/1855
Opened: 6 Apr 2017
*Age: 115 days*
Status: Ready for review. *No comments.*
Comments: Small but important build fix. Should be reviewed.

#1863: Build: use default sysconfdir instead of /etc
Link: https://github.com/BOINC/boinc/pull/1863
Opened: 11 Apr 2017
*Age: 110 days*
Status: Ready for review. *No comments.*
Comments: Small but important build fix. Should be reviewed.

#1865: Build: test if hard links are really working
Link: https://github.com/BOINC/boinc/pull/1865
Opened: 11 Apr 2017
*Age: 110 days*
Status: Ready for review. *No comments.*
Comments: *This is a fix for an issue which was originally found 4(!) years
ago.*

#1878: SCHED_BATCH -> SCHED_IDLE
Link: https://github.com/BOINC/boinc/pull/1878
Opened: 22 Apr 2017
*Age: 99 days*
Status: Ready for review.
Comments: This pull request changes a bit a task priority under linux.
Should be reviewed and some decision should be made.

#1886: Build Windows: Add AppVeyor support for Windows CI builds.
Link: https://github.com/BOINC/boinc/pull/1886
Opened: 25 Apr 2017
*Age: 96 days*
Status: Ready for review.
Comments: This pull request add another CI (AppVeyor) service to run
Windows build. This is very helpful as well as #1821 (Mac CI build).

#1895: Move 'strlen' function outside of the loop
Link: https://github.com/BOINC/boinc/pull/1895
Opened: 2 May 2017
*Age: 89 days*
Status: Ready for review.
Comments: This pull request affects performance of the boinc application
and makes a code a little bit cleaner. It doesn't decrease readability of
the code. Not all this actions can be made by compiler.

#1946: Modifications that have been included in the version that is
currently used by LHC@home.
Link: https://github.com/BOINC/boinc/pull/1946
Opened: 20 Jun 2017
*Age: 40 days.*
Status: In progress.
Comments: Pull request is not ready for merge now.

#1956: Lib: fix escape_url() for UTF8 encoded strings
Link: https://github.com/BOINC/boinc/pull/1956
Opened: 26 Jun 2017
*Age: 33 days*
Status: Has conflicts.
Comments: This pull request was reintroduced by David Anderson in
https://github.com/BOINC/boinc/commit/ff7633a65def104b81a82b839f98fbe4b383f42f
. This pull request can be closed as from my POV but need review before
this.

#1968: db_purge: use zlib
Link: https://github.com/BOINC/boinc/pull/1968
Opened: 11 Jul 2017
*Age: 19 days*
Status: Build is failed.
Comments: Should be reviewed. In case of positive decision Travis build
configuration should be changed to prevent build fail.

#1977: winbuild: add keyword.{h,cpp} to VS2013 solution
Link: https://github.com/BOINC/boinc/pull/1977
Opened: 17 Jul 2017
*Age: 12 days*
Status: Ready for review. *No comments.*
Comments: Important pull request to fix build on VS2013.

#1978: client: add task name and tweak postponed message
Link: https://github.com/BOINC/boinc/pull/1978
Opened: 17 Jul 2017
*Age: 12 days*
Status: Ready for review. *No comments.*
Comments: This pull request add some helpful information to logs. Should be
reviewed.

#1979: client/lib: don't flush stdout and stderr in main loop
Link: https://github.com/BOINC/boinc/pull/1979
Opened: 17 Jul 2017
*Age: 12 days*
Status: Ready for review.
Comments: Should be reviewed.

#1980: lib: fix out of bounds write in get_real_executable_path
Link: https://github.com/BOINC/boinc/pull/1980
Opened: 17 Jul 2017
*Age: 12 days*
Status: Ready for review. *No comments.*
Comments: Should be reviewed. Very important fix of a quite dangerous bug.

#1933: Drupal: Added a second link to rules-and-policies
Link: https://github.com/BOINC/boinc/pull/1993
Opened: 26 Jul 2017
*Age: 3 days*
Status: Ready.
Comments: I'm sure this will be reviewed and merged by @tristanolive in a
several days.

#1995: Drupal: Forum top links are now placed into two columns.
Link: https://github.com/BOINC/boinc/pull/1995
Opened: 26 Jul 2017
*Age: 3 days*
Status: Ready.
Comments: I'm sure this will be reviewed and merged by @tristanolive in a
several days.

#1997: Drupal: Fixed bug where anonymous users, who are not logged in,
could …
Link: https://github.com/BOINC/boinc/pull/1997
Opened: 28 Jul 2017
*Age: 1 day*
Status: Ready.
Comments: I'm sure this will be reviewed and merged by @tristanolive in a
several days.

22 open pull requests is not a good sign for new people who can be
interested in BOINC development. So please guys lets make the project looks
a little bit better.

Thank you for your attention.

Best regards,
Vitalii Koshura
_______________________________________________
boinc_dev mailing list
boinc_dev@ssl.berkeley.edu
https://lists.ssl.berkeley.edu/mailman/listinfo/boinc_dev
To unsubscribe, visit the above URL and
(near bottom of page) enter your email address.

Reply via email to