Re: Διαχείριση pull-requests σε mirror repository

[Giorgos Keramidas]

Η ημερομηνία που έγινε κάτι commit είναι σημαντική πληροφορία, οπότε θά
λεγα να το ψάξεις λίγο ακόμα.

Αυτό που λες συμβαίνει μόνο αν το original patch δεν έχει 'Date' header,
π.χ. δες το σωστό commit date στο 'hg import'ed patch παρακάτω:

1206 11:18 gkeramidas at gkeramidas-mbp:/tmp $ *hg init demo*

1206 11:18 gkeramidas at gkeramidas-mbp:/tmp $ *cd demo*

1206 11:18 gkeramidas at gkeramidas-mbp:/tmp/demo $ *echo foo > foo.txt*

1206 11:18 gkeramidas at gkeramidas-mbp:/tmp/demo $
*hg commit -Am 'add foo'*adding foo.txt

1206 11:18 gkeramidas at gkeramidas-mbp:/tmp/demo $
*echo foo foo > foo.txt*
1206 11:18 gkeramidas at gkeramidas-mbp:/tmp/demo $
*hg commit -Am 'more foo'*
1206 11:19 gkeramidas at gkeramidas-mbp:/tmp/demo $
*hg log*changeset:   1:23a0efb4c8fc
tag:         tip
user:        Giorgos Keramidas <giorgos at box.com>

*date:        Fri Dec 06 11:19:04 2013 -0800                 *summary:
more foo

changeset:   0:611fa57517da
user:        Giorgos Keramidas <giorgos at box.com>
date:        Fri Dec 06 11:18:50 2013 -0800
summary:     add foo

1206 11:19 gkeramidas at gkeramidas-mbp:/tmp/demo $ *hg export 1 >
/tmp/foo-patch.diff*

1206 11:19 gkeramidas at gkeramidas-mbp:/tmp/demo $
*hg strip 1*1 files updated, 0 files merged, 0 files removed, 0 files
unresolved
saved backup bundle to
/private/tmp/demo/.hg/strip-backup/23a0efb4c8fc-backup.hg

1206 11:19 gkeramidas at gkeramidas-mbp:/tmp/demo $
*hg log*changeset:   0:611fa57517da
tag:         tip
user:        Giorgos Keramidas <giorgos at box.com>
date:        Fri Dec 06 11:18:50 2013 -0800
summary:     add foo

1206 11:19 gkeramidas at gkeramidas-mbp:/tmp/demo $
*head -5 /tmp/foo-patch.diff*# HG changeset patch
# User Giorgos Keramidas <giorgos at box.com>


*# Date 1386357544 28800                  #      Fri Dec 06 11:19:04 2013
-0800    *# Node ID 23a0efb4c8fc7eb273f7e59f2b4b92e0f3372261


1206 11:19 gkeramidas at gkeramidas-mbp:/tmp/demo $ *hg import
/tmp/foo-patch.diff*
applying /tmp/foo-patch.diff

1206 11:19 gkeramidas at gkeramidas-mbp:/tmp/demo $ *hg log*
changeset:   1:23a0efb4c8fc
tag:         tip
user:        Giorgos Keramidas <giorgos at box.com>

*date:        Fri Dec 06 11:19:04 2013 -0800     *summary:     more foo

changeset:   0:611fa57517da
user:        Giorgos Keramidas <giorgos at box.com>
date:        Fri Dec 06 11:18:50 2013 -0800
summary:     add foo



2013/12/6 George Notaras <gnot at g-loaded.eu>

> On 01/12/2013 09:15, George Notaras wrote:
>
> > Αν στο URL της σελίδας του pull request προσθέσουμε την κατάληξη
> > '.patch', τότε εμφανίζεται ένα plain text περιεχόμενο με όλες τις
> > αλλαγές σε μορφή patch. Τώρα, γιατί αυτό θα έπρεπε να υπάρχει ως
> > πληροφορία σε ένα blog post με 'Github Secrets' και όχι ως προφανής
> > επιλογή στη σελίδα του pull request είναι αξιοπερίεργο!
> >
> > Οπότε, αν πάρουμε αυτό το patch και το κάνουμε apply στο hg repository,
> > τότε το αυτοματοποιημένο mirroring θα μεταφέρει τις αλλαγές από το Α στο
> > Β χωρίς να υπάρξει κανένα πρόβλημα με την αλληλουχία των changesets.
> > Επίσης, αν ο contributor έχει λογαριασμό και στις δύο υπηρεσίες, τότε
> > ελπίζω πως τα commits θα δείχνουν το σωστό συγγραφέα, οπότε πιστεύω πως
> > λύνεται και αυτό.
> >
> > Δηλαδή, θεωρητικά πάντα, μπορεί να δέχεται κανείς pull requests και στο
> > github mirror με την προϋπόθεση ότι δεν θα τις κάνει ποτέ merge,
> > αλλά θα τις μεταφέρει ως patch στο κύριο repository (hg).
>
> Δοκίμασα τα παραπάνω στην πράξη και το αποτέλεσμα ήταν το αναμενόμενο
> επιθυμητό. Έχω την εντύπωση ότι το merge&close ή το σκέτο close των pull
> requests δεν έχει και τόσο μεγάλη πρακτική σημασία.
>
> Το μόνο αρνητικό που διαπίστωσα στην όλη διαδικασία ήταν ότι το `hg
> import foo.patch` βάζει τωρινή ημερομηνία στα changesets αντί για την
> ημερομηνία που περιλαμβάνεται στο Patch για το κάθε changeset.
>
>
>
> --
> linux-greek-users mailing list -- http://lists.hellug.gr
>



-- 
Giorgos Keramidas ∙ gkeramidas at gmail.com
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.hellug.gr/pipermail/linux-greek-users/attachments/20131206/f2003554/attachment.html>