I took a look at the VCL source and have the following comments

INSTALLATION contains references to the Virtual Computing Laboratory at NCSU

In general, there are still a fair number of references to NCSU. For example, in managementnode/lib/VCL/utils.pm:

        # Fix the unityid if non-NCSU account
        if ($request_info{user}{uid} >= 1000000) {
my ($correct_unity_id, $user_domain) = split /@/, $request_info{user} {unityid}; notify($ERRORS{'OK'}, 0, "non-NCSU user found: $request_info{user} {unityid}, $correct_unity_id from $user_domain");
                $request_info{user}{unityid}    = $correct_unity_id;
                $request_info{user}{STANDALONE} = 1;
        }

There are a lot of AUTHOR tags. The ASF Board recommends that projects not use author tags in source code. AFAIK, this is a "recommendation" not "policy". And this is a bit of a contentious issue to some. However, I recommend following the boards recommendation. I wouldn't vote -1 simply because of author tags. However, would like to see the community set a direction on their use.

A lot of modules contain:

=head1 COPYRIGHT

 Apache VCL incubator project
 Copyright 2009 The Apache Software Foundation

 This product includes software developed at
 The Apache Software Foundation (http://www.apache.org/).

Since the source already contains recommended source license headers, are they really necessary? They're something additional to maintain. NOTICE file should contain the copyright statement for Apache VCL.

The following files do not contain a source license header. Most look like they syntactically could contain a license header and most look like they should (INSTALLATION is debatable)...

./INSTALLATION
./managementnode/lib/VCL/.perltidyrc
./managementnode/lib/VCL/Module/Provisioning/esx.README
./managementnode/tools/default.tmpl
./managementnode/tools/Windows/Utilities/Sysprep/i386/$oem$/cmdlines.txt
./managementnode/tools/Windows_Server_2003/Utilities/Sysprep/sysprep.inf
./managementnode/tools/Windows_XP/Utilities/Sysprep/sysprep.inf
./web/testsetup.php
./web/shibauth/.htaccess
./web/shibauth/index.php

IIUC, there is no "build" process. Using VCL is a matter of following the installation instructions. It may be extremely apparent to everyone, but I think it best to point out somewhere (INSTALLATION and or RELEASE_NOTES) that the prerequisites are under their own licensing terms (i.e. not all are covered by an apache license).

--kevan



Reply via email to