Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ls: fix incomplete listing for specific prefix #1849

Merged
merged 1 commit into from
Oct 5, 2016

Conversation

balamurugana
Copy link
Member

Previously it was failing like below.

$ mc ls --incomplete myminio/mybucket
[2016-09-16 09:48:57 PDT] 576MiB 1.iso

$ mc ls --incomplete myminio/mybucket/1.iso
mc: <ERROR> Unable to initialize target ‘myminio/mybucket/1.iso’. Object key is missing, object key cannot be empty

This patch fixes this issue.

Previously it was failing like below.

```
$ mc ls --incomplete myminio/mybucket
[2016-09-16 09:48:57 PDT] 576MiB 1.iso

$ mc ls --incomplete myminio/mybucket/1.iso
mc: <ERROR> Unable to initialize target ‘myminio/mybucket/1.iso’. Object key is missing, object key cannot be empty
```

This patch fixes this issue.
@codecov-io
Copy link

codecov-io commented Oct 4, 2016

Current coverage is 8.72% (diff: 3.70%)

Merging #1849 into master will decrease coverage by 0.02%

@@            master     #1849   diff @@
========================================
  Files           77        77          
  Lines         6051      6066    +15   
  Methods          0         0          
  Messages         0         0          
  Branches         0         0          
========================================
  Hits           529       529          
- Misses        5391      5406    +15   
  Partials       131       131          

Powered by Codecov. Last update f03aaef...1c7aebd

@harshavardhana harshavardhana merged commit 75d5578 into minio:master Oct 5, 2016
@balamurugana balamurugana deleted the stat-fix branch October 18, 2016 00:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants