Re: Bundles in Agora

[Juan Pablo Carbajal]

On Sat, Oct 27, 2012 at 10:51 PM, Wendy Liu <address@hidden> wrote:
> On 22 October 2012 06:09, c. <address@hidden> wrote:
>>
>> Hi Wendy,
>>
>> I just prepared new releases for some packages in Octave-Forge and I tried
>> to upload them
>> to Agora in order to test the bundle-submission system.
>>
>> Here are some comments you may (or may not) find useful to improve its
>> functionality.
>>
>> 1) "Description"
>>    Bundles formatted as Octave loadable packages do include a
>> "DESCRIPTION" file,
>>    wouldn't it be possible to parse that directly as it is currently done
>> to create
>>    the package page on Octave-Forge?
>
>
> Done, you can see a demo here:
> http://agora.dellsystem.me/bundles/testing/description-test Let me know if
> that's not what you had in mind.
>
>>
>>
>> 2) Docs
>>    When uploading a package on Octave-Forge we include a set of html files
>> with the package
>>    documentation that are automatically generated using the
>> "generate_package_html" function.
>>    Wouldn't it be possible to allow users to upload such files on Agora as
>> well and show them
>>    instead of the bundle file listing?
>
>
> I'll look into this, thanks for the suggestion.
>
>>
>>
>> 3) Browsing bundle contents
>>    I don't think the ability to browse the contents of all the files in a
>> bundle makes much
>>    sense, it would be better to just show the description, index, license
>> and possibly
>>    the HTML docs.
>
>
> The thing is, the ability to browse the contents of a file is already
> present, and I think it's more useful to have this feature present than to
> not, just in case someone ever wants to look at a file without having to
> download the package, so I would prefer to keep this feature. It's also
> handy if you ever want to link to a specific line in a file (when reporting
> a bug, for instance).
>
>> 3) "Name" field.
>>    It would be nice to add a warning under the field title specifying the
>> acceptable
>>    format for the field to avoid trial-and-error
>
>
> This has been done, agora.octave.org hasn't been updated with it yet though.
>
>>
>>
>> 4) "Paste it" button.
>>    Shouldn't this read "Upload" in this context?
>
>
> I've changed it to "Save bundle", thanks for pointing that out. The same
> form is used when editing a bundle (and it's not necessary to upload a new
> version in that form) so I tried to pick something that would work in all
> cases.
>
>>
>> 5) Formatted and unformatted bundles.
>>    These suggestions are biased towards the case were bundles being
>> uploaded are in the format
>>    required by the Octave package manager ("pkg.m"), you might want to add
>> a flag in the upload
>>    page that allows to specify whether the bundle peing uploaded is
>> formatted according to this
>>    format or not and behave differently in the two cases.
>
>
> That's a good idea. I've added this as a checkbox.
>
>>
>> These are just my two cents, others may think differently.
>> Anyways, starts to look very nice, keep up the very good work!
>> Thanks,
>> Carlo

Thank for all the changes Wendy!

I agree with your comment on "3) Browsing bundle contents"