Em 23-05-2011 06:11, Christian Johansen escreveu:
Hi Rodrigo,

I just updated the official Rails 3.1 branch. I feel uneasy about the rebasing stuff, so we're proposing a slightly different way continuing forward: You continue to rebase master into your branch, and then we will merge your branch to the official Rails 3.1 branch. Sounds ok?

I guess so, we'll figure out.

I'm advancing in the migration to Devise in the temporary branch (gitorious-3.1-devise-temp). Currently, it is necessary to add support for OpenID and Basic HTTP authentication.

Also, I need to figure out what is needed for the SSL enforcing tests to pass (I've disabled them temporarily for better filtering the failing tests). But it seems Rails 3.1 may not need the ssl_requirement plugin since there will be a native helper for that:

https://github.com/rails/rails/blob/master/actionpack/lib/action_controller/metal/force_ssl.rb

Ok, this is just a status update. I'll try to work more on this at night.

Best regards,

Rodrigo.

On Thu, May 19, 2011 at 04:06, Rodrigo Rosenfeld Rosas <[email protected] <mailto:[email protected]>> wrote:

    I pushed (forced) my rebased rails-3.1 branch. All tests green.
    Please update yours rails-3.1 branch to reflect it.

    I've pushed an unfinished branch with the migration to Devise
    (rails-3.1-devise-temp). I'm having some problems regarding routes
    and I intend to talk about this with you tomorrow (today for you).

    Best regards, Rodrigo.

    Em 18-05-2011 11:25, Marius Mårnes Mathiesen escreveu:
    On Wed, May 18, 2011 at 2:26 PM, Rodrigo Rosenfeld Rosas
    <[email protected] <mailto:[email protected]>> wrote:

        Em 18-05-2011 03:43, Marius Mårnes Mathiesen escreveu:
        On Mon, May 16, 2011 at 11:38 PM, Christian Johansen
        <[email protected] <mailto:[email protected]>> wrote:

            I'm not entirely sure how to handle this either, but I
            do know I want to keep the rails-3.1 branch up to date
            in case anyone else wants to chip in. As far as I know,
            the branch Marius pushed today is currently a spike, and
            it's not given that will merge it as is (Marius will
            have to confirm this). Maybe Marius has some input on
            the process going forward too.


        Well, I'm trying to keep the search stuff in master, not
        breaking anything. As for rebasing, wouldn't it be best if
        you rebase as you see fit? That's probably the best way to
        avoid diverging the two branches too much?

        Sorry, who is "you" here? And I think we should always rebase
        as soon as master advances for easying the process...

        But I think it would be simpler if there was a single person
        doing the rebase.

        Here is the problem. If I do the rebase, all commits change
        so that the only way to check if the original commits were
        not modified is to re-check all of them.

        When I do the rebase *I* know that I've changed nothing on
        them unless there was some conflict. In that case, I know how
        I managed it, but the other ones would need to re-check them
        all again.

        I'm ok if someone will be responsible for rebasing it since I
        rely on both you. After rebasing I would checkout your rebase
        and than proceed with my commits.


    I have no problems if you do the rebase; it would probably be too
    cumbersome for you to rely on someone else rebasing, don't you
    think? The fact that the commits will change is a potential
    issue, of course, but the output of git log will be a lot better
    if you rebase instead of merging?

        Anyway, I just need to know how we're going to proceed
        regarding this synchronization with master. It is important
        to syncronize as often as possible or it will be much harder
        to do it later on, specially when there are other people
        working in top of rails-3.1 branch.


    I totally agree.


--
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]

Reply via email to