Fix minimal modules bug

The minimal modules bug on PrgEnv-cray was due to an empty array
initializer list with an inferred size, which is apparently illegal.
The fix was as simple as printing out a single null pointer in the
list in this case (something that we already did for a previous
list -- I simply mimicked the logic).
Update interface to string_from_c_string_copy() to consume its argument.

For safety, a routine taking a c_string_copy argument should accept that argument by
reference and set its value to NULL in the caller.  This can prevent double-deletion
errors by enforcing that the actual argument is not reused after it has been adopted.

This change necessitated changing const variables to standard "var" variables, so the
c_string_copy can be overwritten with _stringNull after it has been adopted.

Renamed chpl_free_c_string() as chpl_free_c_string_copy(), since we should never free a
c_string.

Removed wide_string_from_c_string_copy(), since it appears to be unused.
First draft of perf graph url updates
Change signature of chpl_rt_free_c_string to take a c_string_copy instead.

Only the c_string_copy version of the function should be required, but a c_string version
is supplied in the interim as a bridge.  A c_string is "unowned" so it is technically
illegal to free one.

The new string implementation uses c_string as its base type instead of c_string_copy,
because it was created before the distinction was made between "owned" c_string_copy
objects and "unowned" c_string objects.  Rewriting string_rec to use c_string_copy as its
base type is left as a separate task.

TODO: Rewrite string_rec to use c_string_copy as its base type.

###DETAILS
chpl_rt_free_c_string_copy() was rewritten to take its argument by reference and zero out
the referenced c_string_copy pointer when done.  On the Chapel side, the pointer to be
deleted must be passed using ref intent.  In several places, it was necessary to add the
ref intent to an argument or to change "const" to "var" to make a local variable writable.
Merge branch 'robustCStringCopy' of github.com:hildeth/chapel into robustCStringCopy

# Please enter a commit message to explain why this merge is necessary,
# especially if it merges an updated upstream into a topic branch.
#
# Lines starting with '#' will be ignored, and an empty message aborts
# the commit.
Changed leading character of instructions to support grep story counting
Merge branch 'master' of https://github.com/chapel-lang/chapel into fix-minmodules-empty-vmt
Merge branch 'master' of https://github.com/chapel-lang/chapel into fix-minmodules-empty-vmt
Fix counts in nightly emails.

Fix nightly so it correctly get "0" futures, instead of "0]" futures for this
summary:

  [Summary: #Successes = 5021 | #Failures = 4 | #Futures = 0]
Update the baseline suppressions and a .skipif so that --baseline can run without throwing --inline
Merge pull request #832 from thomasvandoren/fix-nightly-counts

Fix counts in nightly emails.

Fix nightly so it correctly get "0" futures, instead of "0]" futures for this
summary:

    [Summary: #Successes = 5021 | #Failures = 4 | #Futures = 0]

r: @lydia-duncan
Update chown/nothing.prediff to check all user groups.

The group assigned to a particular file will not necessarily be the group
reported by `$(id -gn)`. Update nothing.prediff to check that the group
assigned to the file is one of the groups reported by `$(groups)`.

bash-ify the prediff along the way.
Merge pull request #833 from noakesmichael/inline

Support --baseline without --inline

1) Restore the state of the baseline Suppressions file
2) Extend the .skipif for op_equals_using_C_primitive per email thread
3) Update the nightly bash script that manages the baseline run

Tested the suppressions and skipif.
Reviewed: Thomas
Reduce --no-local errors from 121 to fewer.

Turn a few more consts into vars.
Make _cast(type t, ref x: c_string_copy) where t == string take its c_string_copy argument
by reference.
Rewrite beer programs to store c_string_copy results in named variables.
Add beer-submitted.notest, so we do not test the submitted version.
Fix sub_test to not pass numLocales to c_tests
Merge pull request #834 from thomasvandoren/robustify-chown-nothing-test

Update chown/nothing.prediff to check all user groups.

The group assigned to a particular file will not necessarily be the group
reported by $(id -gn). Update nothing.prediff to check that the group
assigned to the file is one of the groups reported by $(groups).

bash-ify the prediff along the way.
Second update to graph url

This is part of a semi large effort to clean up how we store options in the url
for our graphs. This part adds support for having the suites in the url and
switched from having graphs represented with their  checkbox ids to having
graphs be represented by a normalized title. This makes it easy to have a URL
that is valid for a long period of time. Previously anytime the number of
graphs changed the URL would become outdated. This allows the URL for
individual graphs to be good until any graph title changes (happens
infrequently) and it makes it so we can have URL's for suites. The suites is
useful in hyperlinks that we want around long term such as a link in a blog.

This adds functionality to have the suite in the url and a bunch of helper
routines that were required as part of that and changes individual graphs to be
stored by title. It also adds a bunch of comments and does some cleanup.
Merge pull request #835 from Kyle-B/kb-num-locales-c-tests

Change sub_test to not pass numLocales to C tests
Add gnu.darwin test script.

Will test gnu compiler on darwin platform nightly.
fix comment