Re: imread and imfinfo updates 33613 37369

[Carnë Draug]

On 5 December 2012 16:54, Mark Wistrom <address@hidden> wrote:
> -----Original Message-----
> From: address@hidden [mailto:address@hidden On Behalf Of Carnë
> Draug
> Sent: Wednesday, December 05, 2012 7:50 AM
> To: Mark Wistrom
> Cc: address@hidden
> Subject: Re: imread and imfinfo updates 33613 37369
>
> On 3 December 2012 22:04, Mark Wistrom <address@hidden> wrote:
>> first checkin.  still learning ropes.
>> changes bring Octave to be a little more compatible with Matlab.
>
> Hi Mark
>
> you must have attached the wrong changeset. It doesn't change any actual
> code. Also, you better attach changesets to the respective bug report on the
> bug tracker. Changesets on the mailing list tend to get lost way too easily.
>
> Attach your patch to one of the bugs you mentioned mentioning that it also
> fixes some other bug. If you can make two patches, one for each bugfix, even
> better. And if you put the bug number at the end of the commit message,
> perfect. See http://wiki.octave.org/Commit_message_guidelines
>
> I spent some time looking over the checkin procedure, and it was a bit
> confusing to me. I'll go ahead and hack on it some more. Seems the best way
> for me to learn anyway. At the least, if something seems broken or a bit
> confusing, I could work on the instructions, (once I figure them out).
>
> Thanks for your time.
>
> Mark

Always make "reply to all" to include the mailing list and add your
reply at the bottom of the mail. Ask in the mailing list if you have a
problem preparing the changeset, or if the instructions are confusing
in anyway. We want it to be easy for others to contribute.

Carnë