On Thu, May 6, 2010 at 2:59 PM, sebb <seb...@gmail.com> wrote:
> On 06/05/2010, Paul Querna <p...@querna.org> wrote:
>> On Thu, May 6, 2010 at 2:24 PM, Paul Querna <p...@querna.org> wrote:
>>  > On Thu, May 6, 2010 at 2:20 PM, sebb <seb...@gmail.com> wrote:
>>  >> On 06/05/2010, Paul Querna <p...@querna.org> wrote:
>>  >>> The Apache Libcloud project is currently voting on our second release,
>>  >>>  0.3.0.  As required by Incubator policy, we need IPMC approval for
>>  >>>  this release.  There is a concurrent release vote ongoing on
>>  >>>  <libcl...@incubator.apache.org>.
>>  >>>
>>  >>>  The current (passing) RAT output can be seen at:
>>  >>>  <http://ci.apache.org/projects/libcloud/rat-output.txt>
>>  >>
>>  >> Which does not include many of the files in the release archive - e.g
>>  >> none of the fixtures are included.
>>  >>
>>  >> I checked a few of these, and none of the json or XML files had AL 
>> headers.
>>  >
>>  > These are in the .ratignore, because all of these are test fixtures,
>>  > returning raw data from API providers for the test cases, so adding AL
>>  > headers would invalidate the tests.
>>
>
> Is that true of *all* the fixture files?

Yes.  They are all used by our MockHTTPLayer as raw response bodies --
adding license blocks to them would make them different from what the
providers return.

>> To be clear, the tarball doesn't include the .ratignore file, so I'm
>>  not sure running RAT over the generated tarballs will produce the
>>  expected results.
>>
>>  The best test would be to run
>>  svn checkout https://svn.apache.org/repos/asf/incubator/libcloud/tags/0.3.0
>>  libcloud-0.3.0
>>
>>  and run RAT from there.
>>
>>  As documented in HACKING, the actual release tarballs are generated by
>>  dist/release.sh, which doesn't include developer releated files like
>>  .ratignore and .gitignore.
>
> There is still the problem of the incorrect NOTICE file.

I've fixed it in trunk:
http://svn.apache.org/repos/asf/incubator/libcloud/trunk/NOTICE

With just the NOTICE issue, do you maintain a -1 vote?

> Also, I could not run the test suite:
>
>> python setup.py test
> running test
> Traceback (most recent call last):
>  File "setup.py", line 98, in <module>
>    'Topic :: Software Development :: Libraries :: Python Modules'
>  File "c:\python26\lib\distutils\core.py", line 152, in setup
>    dist.run_commands()
>  File "c:\python26\lib\distutils\dist.py", line 975, in run_commands
>    self.run_command(cmd)
>  File "c:\python26\lib\distutils\dist.py", line 995, in run_command
>    cmd_obj.run()
>  File "setup.py", line 45, in run
>    tests = TestLoader().loadTestsFromNames(testfiles)
>  File "c:\python26\lib\unittest.py", line 613, in loadTestsFromNames
>    suites = [self.loadTestsFromName(name, module) for name in names]
>  File "c:\python26\lib\unittest.py", line 576, in loadTestsFromName
>    module = __import__('.'.join(parts_copy))
>  File "D:\ReleaseCheck\apache-libcloud-0.3.0\test\__init__.py", line
> 19, in <module>
>    from libcloud.base import Node, NodeImage, NodeLocation
>  File "D:\ReleaseCheck\apache-libcloud-0.3.0\libcloud\base.py", line
> 21, in <module>
>    from zope import interface
> ImportError: No module named zope

libcloud depends on zope interface for defining and validating that
the python APIs are consistent between drivers:
http://pypi.python.org/pypi/zope.interface
easy_install zope.interface

In addition, to my knowledge, no one actually tests or run libcloud on
windows, so any bug reports there would be great to add to JIRA,

Thanks,

Paul

---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org

Reply via email to