<div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">>  - remove the annoying check of free space<br>
<br>
</div>I hope that's not completely removed (I see Gonzalo's email...). It is<br>
not very effective preventing you from using activities, but it does<br>
prompt users to do something (remove stuff?).<br>
<br>
BTW, 50MB may be too high, some units with 2GB storage, installing<br>
modern builds, end up with 150MB free total :-/<br>
<div class="im"><br></div></blockquote><div><br></div><div>Yes, we agree, but decided not change it in this patch to avoid a long discussion about the size.</div><div>Can be changed in another patch.</div><div><br></div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
>  - create a signal inside sugar-toolkit-gtk3 datastore that is emitted<br>
> when free space is behind<br>
> "sugar3.datastore.datastore.SPACE_THRESHOLD". This check is done every<br>
> time a model is updated or created by the datastore.<br>
<br>
</div>A model? Maybe my lingo is a bit stale... a datastore entry you mean?<br>
<div class="im"><br></div></blockquote><div><br></div><div>Yes. The datastore related code in sugar is separated between the model and the view ;)</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">
>  - this signal can be connected from every activity that wants to<br>
> handle this situation. For example in Browse we are cancelling the<br>
> download in progress when we get that signal<br>
<br>
</div>So there is a bit of confusion here...<br>
<br>
 - When we start the download, if we get Content-Length, we compare it<br>
with disk space available, or with disk space available minus<br>
datastore.SPACE_THRESHOLD?<br></blockquote><div><br></div><div>with availabe - SPACE_THRESHOLD  </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
 - Related: can we ask the datastore to tell us its SPACE_THRESHOLD,<br>
so we don't hardcode it? Ah, I see you have a helper function.<br>
<br>
Personally, I would prefer to preserve a relatively high threshold<br>
(lower than 50MB, but say, 10MB) for the _warnings_. But still allow a<br>
download to complete. I would only cancel a download in Browse if will<br>
leave us with <1MB disk space.<br>
<br></blockquote><div><br></div><div>Well, Manuel proposed that, and I said better not, :( </div><div>but have sense, and can be implemented, </div><div>that is the reason there are a error code, to enable us more specific messages. </div>
<div> </div><div>
<br>Gonzalo</div></div>