Thread: [eclipsedarcs-develop] Push All operation refactoring + Send All Fix.

Status: Alpha

Brought to you by: radoslawg

eclipsedarcs-develop

[eclipsedarcs-develop] Push All operation refactoring + Send All Fix.

From: <rad...@gm...> - 2007-03-14 13:29:25

Attachments: pushall.azip

Hi,
  here comes patches for Push All operation. It works now (if you
select "last repository...").

Also there is trivial error in validating SendAll Dialog which
basically prevented it from working.

Attached file is zip file.

Wed Mar 14 14:19:49 [_\8c_]rodkowoeuropejski czas stand. 2007
Radoslaw Grzanka <rad...@gm...>
  * Trivial error in validating Send All Dialog.

Wed Mar 14 14:15:19 [_\8c_]rodkowoeuropejski czas stand. 2007
Radoslaw Grzanka <rad...@gm...>
  * Pull All Dialog fix.

Wed Mar 14 14:15:01 [_\8c_]rodkowoeuropejski czas stand. 2007
Radoslaw Grzanka <rad...@gm...>
  * Push All Dialog refactoring.

Wed Mar 14 14:14:20 [_\8c_]rodkowoeuropejski czas stand. 2007
Radoslaw Grzanka <rad...@gm...>
  * Push All Operation refactoring to employ *Info model.


Cheers,
  Radek.

Re: [eclipsedarcs-develop] Push All operation refactoring + Send All Fix.

From: Leif F. <lei...@go...> - 2007-03-16 06:08:19

SGkgUmFkZWssCgpUaGFua3MgOi0pIEknbSBob3dldmVyIG5vdCBhYmxlIHRvIGFwcGx5IHRoaXMg
bGFzdCBvbmUuIEkgZ2V0Cgpbc25pcF0KQzpcd29ya3NwYWNlc1xlY2xpcHNlZGFyY3M+ZGFyY3Mg
YXBwbHkgLS1pbnRlcmFjdGl2ZSBwdXNoYWxsLnBhdGNoCmRhcmNzOiBDYW5ub3QgYXBwbHkgdGhp
cyBwYXRjaCBidW5kbGUsIHNpbmNlIHdlJ3JlIG1pc3Npbmc6ClRodSBNYXIgIDggMTE6Mjk6MTIg
V2VzdGV1cm9w9WlzY2hlIE5vcm1hbHplaXQgMjAwNyAgUmFkb3NsYXcgR3J6YW5rYSA8CnJhZG9z
bGF3Z0BnbWFpbC5jb20+CiAgKiBGaXggZm9yIGJ1dCAjOCAoQWRkIG9wZXJhdGlvbikKICBUaGUg
YWRkIG9wZXJhdGlvbiBpcyBub3cgZGlzYWJsZWQgaW4gZXZlcnkgY2FzZSB3aGVyZSBpdCB3b3Vs
ZCBub3QgYWRkCmFueXRoaW5nIHRvIHRoZSByZXBvc2l0b3J5LgogICoqKkRBUkNTKioqCltzbmFw
XQoKSXQgc2VlbXMgdGhlcmUgaXMgYSBwYXRjaCBtaXNzaW5nIGluIHRoZSBidW5kbGUsIG9yIGVs
c2UgZGlkIEkgbWlzcwpzb21ldGhpbmc/CgpUaGFua3MgJiYgY2lhbywKTGVpZgoKT24gMy8xNC8w
NywgUmFkb3P5YXcgR3J6YW5rYSA8cmFkb3NsYXdnQGdtYWlsLmNvbT4gd3JvdGU6Cj4KPiBIaSwK
PiAgIGhlcmUgY29tZXMgcGF0Y2hlcyBmb3IgUHVzaCBBbGwgb3BlcmF0aW9uLiBJdCB3b3JrcyBu
b3cgKGlmIHlvdQo+IHNlbGVjdCAibGFzdCByZXBvc2l0b3J5Li4uIikuCj4KPiBBbHNvIHRoZXJl
IGlzIHRyaXZpYWwgZXJyb3IgaW4gdmFsaWRhdGluZyBTZW5kQWxsIERpYWxvZyB3aGljaAo+IGJh
c2ljYWxseSBwcmV2ZW50ZWQgaXQgZnJvbSB3b3JraW5nLgo+Cj4gQXR0YWNoZWQgZmlsZSBpcyB6
aXAgZmlsZS4KPgo+IFdlZCBNYXIgMTQgMTQ6MTk6NDkgW19cOGNfXXJvZGtvd29ldXJvcGVqc2tp
IGN6YXMgc3RhbmQuIDIwMDcKPiBSYWRvc2xhdyBHcnphbmthIDxyYWRvc2xhd2dAZ21haWwuY29t
Pgo+ICAgKiBUcml2aWFsIGVycm9yIGluIHZhbGlkYXRpbmcgU2VuZCBBbGwgRGlhbG9nLgo+Cj4g
V2VkIE1hciAxNCAxNDoxNToxOSBbX1w4Y19dcm9ka293b2V1cm9wZWpza2kgY3phcyBzdGFuZC4g
MjAwNwo+IFJhZG9zbGF3IEdyemFua2EgPHJhZG9zbGF3Z0BnbWFpbC5jb20+Cj4gICAqIFB1bGwg
QWxsIERpYWxvZyBmaXguCj4KPiBXZWQgTWFyIDE0IDE0OjE1OjAxIFtfXDhjX11yb2Rrb3dvZXVy
b3BlanNraSBjemFzIHN0YW5kLiAyMDA3Cj4gUmFkb3NsYXcgR3J6YW5rYSA8cmFkb3NsYXdnQGdt
YWlsLmNvbT4KPiAgICogUHVzaCBBbGwgRGlhbG9nIHJlZmFjdG9yaW5nLgo+Cj4gV2VkIE1hciAx
NCAxNDoxNDoyMCBbX1w4Y19dcm9ka293b2V1cm9wZWpza2kgY3phcyBzdGFuZC4gMjAwNwo+IFJh
ZG9zbGF3IEdyemFua2EgPHJhZG9zbGF3Z0BnbWFpbC5jb20+Cj4gICAqIFB1c2ggQWxsIE9wZXJh
dGlvbiByZWZhY3RvcmluZyB0byBlbXBsb3kgKkluZm8gbW9kZWwuCj4KPgo+IENoZWVycywKPiAg
IFJhZGVrLgo+Cj4gLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t
LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLQo+IFRha2UgU3VydmV5cy4gRWFybiBDYXNoLiBJ
bmZsdWVuY2UgdGhlIEZ1dHVyZSBvZiBJVAo+IEpvaW4gU291cmNlRm9yZ2UubmV0J3MgVGVjaHNh
eSBwYW5lbCBhbmQgeW91J2xsIGdldCB0aGUgY2hhbmNlIHRvIHNoYXJlCj4geW91cgo+IG9waW5p
b25zIG9uIElUICYgYnVzaW5lc3MgdG9waWNzIHRocm91Z2ggYnJpZWYgc3VydmV5cy1hbmQgZWFy
biBjYXNoCj4gaHR0cDovL3d3dy50ZWNoc2F5LmNvbS9kZWZhdWx0LnBocD9wYWdlPWpvaW4ucGhw
JnA9c291cmNlZm9yZ2UmQ0lEPURFVkRFVgo+IF9fX19fX19fX19fX19fX19fX19fX19fX19fX19f
X19fX19fX19fX19fX19fX19fCj4gZWNsaXBzZWRhcmNzLWRldmVsb3AgbWFpbGluZyBsaXN0Cj4g
ZWNsaXBzZWRhcmNzLWRldmVsb3BAbGlzdHMuc291cmNlZm9yZ2UubmV0Cj4gaHR0cHM6Ly9saXN0
cy5zb3VyY2Vmb3JnZS5uZXQvbGlzdHMvbGlzdGluZm8vZWNsaXBzZWRhcmNzLWRldmVsb3AKPgo+
Cj4KCgotLSAKTGVpZiBGcmVuemVsCmh0dHA6Ly9sZWlmZnJlbnplbC5kZQpodHRwOi8vY29oYXRv
ZS5ibG9nc3BvdC5jb20K

Re: [eclipsedarcs-develop] Push All operation refactoring + Send All Fix.

From: <rad...@gm...> - 2007-03-16 07:06:07

Attachments: all.azip

MjAwNy8zLzE2LCBMZWlmIEZyZW56ZWwgPGxlaWZmcmVuemVsQGdvb2dsZW1haWwuY29tPjoKPiBI
aSBSYWRlaywKPgo+IFRoYW5rcyA6LSkgSSdtIGhvd2V2ZXIgbm90IGFibGUgdG8gYXBwbHkgdGhp
cyBsYXN0IG9uZS4gSSBnZXQKPgo+IFtzbmlwXQo+IEM6XHdvcmtzcGFjZXNcZWNsaXBzZWRhcmNz
PmRhcmNzIGFwcGx5IC0taW50ZXJhY3RpdmUKPiBwdXNoYWxsLnBhdGNoCj4gZGFyY3M6IENhbm5v
dCBhcHBseSB0aGlzIHBhdGNoIGJ1bmRsZSwgc2luY2Ugd2UncmUgbWlzc2luZzoKPiBUaHUgTWFy
ICA4IDExOjI5OjEyIFdlc3RldXJvcMO1aXNjaGUgTm9ybWFsemVpdCAyMDA3ICBSYWRvc2xhdyBH
cnphbmthCj4gPHJhZG9zbGF3Z0BnbWFpbC5jb20+Cj4gICAqIEZpeCBmb3IgYnV0ICM4IChBZGQg
b3BlcmF0aW9uKQo+ICAgVGhlIGFkZCBvcGVyYXRpb24gaXMgbm93IGRpc2FibGVkIGluIGV2ZXJ5
IGNhc2Ugd2hlcmUgaXQgd291bGQgbm90IGFkZAo+IGFueXRoaW5nIHRvIHRoZSByZXBvc2l0b3J5
Lgo+ICAgKioqREFSQ1MqKioKPiBbc25hcF0KCkkgdGhvdWdodCBJJ3ZlIHNlbnQgdGhpcyAtIGFw
cGFyZW50bHkgSSBqdXN0IG1hcmtlZCBpdCBkb25lIG9mCmZseXNwcmF5Li4gQW55d2F5LCBoZXJl
IGFyZSBhbGwgdGhlIHBhdGNoZXMgYWdhaW5zdApodHRwOi8vZWNsaXBzZWRhcmNzLnNmLm5ldC9y
ZXBvCgpUaGFua3MsCiAgUmFkZWsuCg==

[eclipsedarcs-develop] Fwd: Push All operation refactoring + Send All Fix.

From: Leif F. <lei...@go...> - 2007-03-16 07:17:12

cnJyZ2ggLi4uIGFnYWluIHRvIHRoZSB3cm9uZyBtYWlsIGFkZHJlc3MgOy0pCgpTb3JyeSAmJiBj
aWFvLApMZWlmCgotLS0tLS0tLS0tIEZvcndhcmRlZCBtZXNzYWdlIC0tLS0tLS0tLS0KRnJvbTog
TGVpZiBGcmVuemVsIDxsZWlmZnJlbnplbEBnb29nbGVtYWlsLmNvbT4KRGF0ZTogTWFyIDE2LCAy
MDA3IDg6MTYgQU0KU3ViamVjdDogUmU6IFtlY2xpcHNlZGFyY3MtZGV2ZWxvcF0gUHVzaCBBbGwg
b3BlcmF0aW9uIHJlZmFjdG9yaW5nICsgU2VuZApBbGwgRml4LgpUbzogUmFkb3P5YXcgR3J6YW5r
YSA8cmFkb3NsYXdnQGdtYWlsLmNvbT4KCkp1cHAsIG5vdyBpdCB3ZW50IHRocm91Z2ggOi0pCgpU
aGFua3MgJiYgY2lhbywKTGVpZgoKT24gMy8xNi8wNywgUmFkb3P5YXcgR3J6YW5rYSA8cmFkb3Ns
YXdnQGdtYWlsLmNvbSA+IHdyb3RlOgoKPiAyMDA3LzMvMTYsIExlaWYgRnJlbnplbCA8IGxlaWZm
cmVuemVsQGdvb2dsZW1haWwuY29tPjoKPiA+IEhpIFJhZGVrLAo+ID4KPiA+IFRoYW5rcyA6LSkg
SSdtIGhvd2V2ZXIgbm90IGFibGUgdG8gYXBwbHkgdGhpcyBsYXN0IG9uZS4gSSBnZXQKPiA+Cj4g
PiBbc25pcF0KPiA+IEM6XHdvcmtzcGFjZXNcZWNsaXBzZWRhcmNzPmRhcmNzIGFwcGx5IC0taW50
ZXJhY3RpdmUKPiA+IHB1c2hhbGwucGF0Y2gKPiA+IGRhcmNzOiBDYW5ub3QgYXBwbHkgdGhpcyBw
YXRjaCBidW5kbGUsIHNpbmNlIHdlJ3JlIG1pc3Npbmc6Cj4gPiBUaHUgTWFyICA4IDExOjI5OjEy
IFdlc3RldXJvcPVpc2NoZSBOb3JtYWx6ZWl0IDIwMDcgIFJhZG9zbGF3IEdyemFua2EKPiA+IDwg
cmFkb3NsYXdnQGdtYWlsLmNvbT4KPiA+ICAgKiBGaXggZm9yIGJ1dCAjOCAoQWRkIG9wZXJhdGlv
bikKPiA+ICAgVGhlIGFkZCBvcGVyYXRpb24gaXMgbm93IGRpc2FibGVkIGluIGV2ZXJ5IGNhc2Ug
d2hlcmUgaXQgd291bGQgbm90IGFkZAo+ID4gYW55dGhpbmcgdG8gdGhlIHJlcG9zaXRvcnkuCj4g
PiAgICoqKkRBUkNTKioqCj4gPiBbc25hcF0KPgo+IEkgdGhvdWdodCBJJ3ZlIHNlbnQgdGhpcyAt
IGFwcGFyZW50bHkgSSBqdXN0IG1hcmtlZCBpdCBkb25lIG9mCj4gZmx5c3ByYXkuLiBBbnl3YXks
IGhlcmUgYXJlIGFsbCB0aGUgcGF0Y2hlcyBhZ2FpbnN0Cj4gaHR0cDovL2VjbGlwc2VkYXJjcy5z
Zi5uZXQvcmVwbwo+Cj4gVGhhbmtzLAo+ICAgUmFkZWsuCj4KPiAtLS0tLS0tLS0tLS0tLS0tLS0t
LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tCj4g
VGFrZSBTdXJ2ZXlzLiBFYXJuIENhc2guIEluZmx1ZW5jZSB0aGUgRnV0dXJlIG9mIElUCj4gSm9p
biBTb3VyY2VGb3JnZS5uZXQncyBUZWNoc2F5IHBhbmVsIGFuZCB5b3UnbGwgZ2V0IHRoZSBjaGFu
Y2UgdG8gc2hhcmUKPiB5b3VyCj4gb3BpbmlvbnMgb24gSVQgJiBidXNpbmVzcyB0b3BpY3MgdGhy
b3VnaCBicmllZiBzdXJ2ZXlzLWFuZCBlYXJuIGNhc2gKPiBodHRwOi8vd3d3LnRlY2hzYXkuY29t
L2RlZmF1bHQucGhwP3BhZ2U9am9pbi5waHAmcD1zb3VyY2Vmb3JnZSZDSUQ9REVWREVWCj4gX19f
X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KPiBlY2xpcHNlZGFy
Y3MtZGV2ZWxvcCBtYWlsaW5nIGxpc3QKPiBlY2xpcHNlZGFyY3MtZGV2ZWxvcEBsaXN0cy5zb3Vy
Y2Vmb3JnZS5uZXQKPiBodHRwczovL2xpc3RzLnNvdXJjZWZvcmdlLm5ldC9saXN0cy9saXN0aW5m
by9lY2xpcHNlZGFyY3MtZGV2ZWxvcAo+Cj4KPgoKCi0tIApMZWlmIEZyZW56ZWwKaHR0cDovL2xl
aWZmcmVuemVsLmRlCmh0dHA6Ly9jb2hhdG9lLmJsb2dzcG90LmNvbQoKCi0tIApMZWlmIEZyZW56
ZWwKaHR0cDovL2xlaWZmcmVuemVsLmRlCmh0dHA6Ly9jb2hhdG9lLmJsb2dzcG90LmNvbQo=

Re: [eclipsedarcs-develop] Push All operation refactoring + Send All Fix.

From: Leif F. <lei...@go...> - 2007-03-17 09:08:31

SGkgUmFkZWssCgp0aGlzIGxvb2tzIGNvb2wgYWxyZWFkeSA6LSkgSSdtIGhhdmluZyBhIGZldyB0
cm91YmxlcyB1bmRlciBXaW5kb3dzLCBtb3N0bHkKcmVsYXRlZCB0byBmaWxlIHBhdGhzIGV0Yy4g
SSB0aGluayBJIHdpbGwgdHJ5IHRvIGZpeCB0aGVtIG15c2VsZiBhbmQgc2VuZAp5b3UgYSBwYXRj
aCBzbyB0aGF0IHlvdSBjYW4gdmVyaWZ5IHRoZXkgZG9uJ3QgYnJlYWsgYW55dGhpbmcgdW5kZXIg
TGludXggOy0pCgpIZXJlIGFyZSBhIGZldyBtb3JlIHN1Z2dlc3Rpb25zOgoKLSBJIHRoaW5rIGl0
IGdlbmVyYWxseSBoYXJkIHRvIHNlZSB3aGF0IG9uZSBoYXMgdG8gZW50ZXIgZm9yIGEgcmVwbyBs
b2NhdGlvbgooc2VlbXMgaXQgbXVzdCBiZSBhIHZhbGlkIFVSSSwgYnV0IEkgb25seSBmaWd1cmVk
IHRoYXQgb3V0IGJlY2F1c2UgSSBzYXcgdGhlCnN0YWNrdHJhY2VzIHdoZW4gc29tZXRoaW5nIHdl
bnQgd3JvbmcgOy0pLiBTbyBJIHRoaW5rIGl0IHdvdWxkIGJlIGdvb2QgaWYgd2UKY291bGQgcmUt
dXNlIHRoZSByZXBvIHNlbGVjdGlvbiBkaWFsb2cgZnJvbSB0aGUgYnJvd3NlciBwZXJzcGVjdGl2
ZSwgd2hlcmUKdGhlIHVzZXIgY2FuIGJyb3dzZSB0byBhIGZvbGRlciBhbmQgaXQgaXMgY29udmVy
dGVkIHRvIGEgVVJJIGluIHRoZQpiYWNrZ3JvdW5kLgoKLSBUaGVyZSBpcyBhIHllbGxvdyB3YXJu
aW5nIGljb24gZm9yIGFuIGludmFsaWQgcmVwbyBsb2NhdGlvbi4gTWF5YmUgd2UKc2hvdWxkIGhh
dmUgc29tZSB3YXJuaW5nIG1lc3NhZ2UgYWxzbyB0aGF0IGdpdmVzIGEgYml0IG1vcmUgaW5mbz8g
KEUuZy4KJ2JsYWJsYWJsYSBpcyBub3QgYSB2YWxpZCByZXBvIGxvY2F0aW9uJyBvciAnYmxhYmxh
YmxhIGRvZXMgbm90IGV4aXN0JyBvcgpzb21ldGhpbmcgbGlrZSB0aGF0KQoKLSBWZXJ5IG1pbm9y
OiBvbiB0aGUgcHVzaC1hbGwgZGlhbG9nLCBJIHRoaW5rIEkgd291bGQgYWN0dWFsbHkgcHJlZmVy
IGlmIHRoZQp0ZXh0IHNheXMgJ2RhcmNzIC0tYWxsJyBpbnN0ZWFkIG9mICdkYXJjcyAtYScuIFRo
YXQgd291bGQgYmUgZWFzaWVyIHRvCnVuZGVyc3RhbmQgdG8gdXNlcnMgd2hvIGRvbid0IGtub3cg
dGhlIGNvbW1hbmQgbGluZSBpbnRlcmZhY2UsIGJlY2F1c2UgaXQgaXMKbW9yZSBzcGVha2luZy4K
CkkgdGhpbmsgYWxzbyB0aGF0IGl0IHdpbGwgaGVscCBpZiB0aGVyZSBpcyBhIHBvc3NpYmlsaXR5
IHRvIGNvcHkgdGhlIHJlcG8KVVJJIGZyb20gYW4gZXhpc3RpbmcgbG9jYXRpb24gKGluIHRoZSBy
ZXBvIGJyb3dzZXIpLiBUaGF0J3Mgbm90IHZlcnkKZGlmZmljdWx0IHRvIGRvLiAoSSdtIHRoaW5r
aW5nIG9mIHNvbWV0aGluZyBsaWtlIHRoZSBQcm9wZXJ0aWVzIGRpYWxvZyB0aGF0CnRoZSBDVlMg
YnJvd3NpbmcgcGVyc3BlY3RpdmUgaGFzIG9uIENWUyByZXBvIGxvY2F0aW9ucywgcGx1cyBwZXJo
YXBzIGEKc2ltaWxhciAnQ29weSB0byBjbGlwYm9hcmQnIGFjdGlvbiBhcyB0aGV5IGRvIGhhdmUu
KSBJIHRoaW5rIEkgY2FuIGRvIHRoYXQKaW4gdGhlIG5leHQgZGF5cywgd2hpY2ggd2lsbCBwcm9i
YWJseSBhbHNvIGJlIGEgZ29vZCB3YXJtLXVwIGV4ZXJjaXNlIGZvcgptZSwgdG8gZ2V0IG9yaWVu
dGVkIGFib3V0IHRoZSBjb2RlIGJhc2UgYWdhaW4gOi0pCgpUaGFua3MgJiYgY2lhbywKTGVpZgoK
T24gMy8xNC8wNywgUmFkb3OzYXcgR3J6YW5rYSA8cmFkb3NsYXdnQGdtYWlsLmNvbT4gd3JvdGU6
Cj4KPiBIaSwKPiAgIGhlcmUgY29tZXMgcGF0Y2hlcyBmb3IgUHVzaCBBbGwgb3BlcmF0aW9uLiBJ
dCB3b3JrcyBub3cgKGlmIHlvdQo+IHNlbGVjdCAibGFzdCByZXBvc2l0b3J5Li4uIikuCj4KPiBB
bHNvIHRoZXJlIGlzIHRyaXZpYWwgZXJyb3IgaW4gdmFsaWRhdGluZyBTZW5kQWxsIERpYWxvZyB3
aGljaAo+IGJhc2ljYWxseSBwcmV2ZW50ZWQgaXQgZnJvbSB3b3JraW5nLgo+Cj4gQXR0YWNoZWQg
ZmlsZSBpcyB6aXAgZmlsZS4KPgo+IFdlZCBNYXIgMTQgMTQ6MTk6NDkgW19cOGNfXXJvZGtvd29l
dXJvcGVqc2tpIGN6YXMgc3RhbmQuIDIwMDcKPiBSYWRvc2xhdyBHcnphbmthIDxyYWRvc2xhd2dA
Z21haWwuY29tPgo+ICAgKiBUcml2aWFsIGVycm9yIGluIHZhbGlkYXRpbmcgU2VuZCBBbGwgRGlh
bG9nLgo+Cj4gV2VkIE1hciAxNCAxNDoxNToxOSBbX1w4Y19dcm9ka293b2V1cm9wZWpza2kgY3ph
cyBzdGFuZC4gMjAwNwo+IFJhZG9zbGF3IEdyemFua2EgPHJhZG9zbGF3Z0BnbWFpbC5jb20+Cj4g
ICAqIFB1bGwgQWxsIERpYWxvZyBmaXguCj4KPiBXZWQgTWFyIDE0IDE0OjE1OjAxIFtfXDhjX11y
b2Rrb3dvZXVyb3BlanNraSBjemFzIHN0YW5kLiAyMDA3Cj4gUmFkb3NsYXcgR3J6YW5rYSA8cmFk
b3NsYXdnQGdtYWlsLmNvbT4KPiAgICogUHVzaCBBbGwgRGlhbG9nIHJlZmFjdG9yaW5nLgo+Cj4g
V2VkIE1hciAxNCAxNDoxNDoyMCBbX1w4Y19dcm9ka293b2V1cm9wZWpza2kgY3phcyBzdGFuZC4g
MjAwNwo+IFJhZG9zbGF3IEdyemFua2EgPHJhZG9zbGF3Z0BnbWFpbC5jb20+Cj4gICAqIFB1c2gg
QWxsIE9wZXJhdGlvbiByZWZhY3RvcmluZyB0byBlbXBsb3kgKkluZm8gbW9kZWwuCj4KPgo+IENo
ZWVycywKPiAgIFJhZGVrLgo+Cj4gLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t
LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLQo+IFRha2UgU3VydmV5cy4gRWFy
biBDYXNoLiBJbmZsdWVuY2UgdGhlIEZ1dHVyZSBvZiBJVAo+IEpvaW4gU291cmNlRm9yZ2UubmV0
J3MgVGVjaHNheSBwYW5lbCBhbmQgeW91J2xsIGdldCB0aGUgY2hhbmNlIHRvIHNoYXJlCj4geW91
cgo+IG9waW5pb25zIG9uIElUICYgYnVzaW5lc3MgdG9waWNzIHRocm91Z2ggYnJpZWYgc3VydmV5
cy1hbmQgZWFybiBjYXNoCj4gaHR0cDovL3d3dy50ZWNoc2F5LmNvbS9kZWZhdWx0LnBocD9wYWdl
PWpvaW4ucGhwJnA9c291cmNlZm9yZ2UmQ0lEPURFVkRFVgo+IF9fX19fX19fX19fX19fX19fX19f
X19fX19fX19fX19fX19fX19fX19fX19fX19fCj4gZWNsaXBzZWRhcmNzLWRldmVsb3AgbWFpbGlu
ZyBsaXN0Cj4gZWNsaXBzZWRhcmNzLWRldmVsb3BAbGlzdHMuc291cmNlZm9yZ2UubmV0Cj4gaHR0
cHM6Ly9saXN0cy5zb3VyY2Vmb3JnZS5uZXQvbGlzdHMvbGlzdGluZm8vZWNsaXBzZWRhcmNzLWRl
dmVsb3AKPgo+Cj4KCgotLSAKTGVpZiBGcmVuemVsCmh0dHA6Ly9sZWlmZnJlbnplbC5kZQpodHRw
Oi8vY29oYXRvZS5ibG9nc3BvdC5jb20K

Re: [eclipsedarcs-develop] Push All operation refactoring + Send All Fix.

From: <rad...@gm...> - 2007-03-18 09:35:21

2007/3/17, Leif Frenzel <lei...@go...>:
> Hi Radek,
>
> this looks cool already :-) I'm having a few troubles under Windows, mostly
> related to file paths etc. I think I will try to fix them myself and send
> you a patch so that you can verify they don't break anything under Linux ;-)
>
> Here are a few more suggestions:
>
> - I think it generally hard to see what one has to enter for a repo location
> (seems it must be a valid URI, but I only figured that out because I saw the
> stacktraces when something went wrong ;-). So I think it would be good if we
> could re-use the repo selection dialog from the browser perspective, where
> the user can browse to a folder and it is converted to a URI in the
> background.

To which dialog window are you refering? I don't recall any, where
user can enter his own repository address

I was about to refine Push/Pull/Send Dialog to allow user to write
address of custom repository and refactor them as they seem to share
much in common.

However, there is a problem with validating indeed. One option is like
you suggested but I don't know how to solve ssh address validation.
IIRC URI for ssh is too complicated to request from user. Ideally I
would like to be able to specify repo address as darcs expects.

> - There is a yellow warning icon for an invalid repo location. Maybe we
> should have some warning message also that gives a bit more info? (E.g.
> 'blablabla is not a valid repo location' or 'blablabla does not exist' or
> something like that)

Are you sure you checked plugin with my patches? I think this icon and
validation is no more. If I missed something then please point me to
it. (I know I am sometimes absent minded)

> - Very minor: on the push-all dialog, I think I would actually prefer if the
> text says 'darcs --all' instead of 'darcs -a'. That would be easier to
> understand to users who don't know the command line interface, because it is
> more speaking.

Yes, agreed.

> I think also that it will help if there is a possibility to copy the repo
> URI from an existing location (in the repo browser). That's not very
> difficult to do. (I'm thinking of something like the Properties dialog that
> the CVS browsing perspective has on CVS repo locations, plus perhaps a
> similar 'Copy to clipboard' action as they do have.) I think I can do that
> in the next days, which will probably also be a good warm-up exercise for
> me, to get oriented about the code base again :-)

What I am working right now on is specifying custom repo location in
Push/Pull and Send dialogs. This will make Eclipsedarcs complete
replacement for command line for me. :)

Next thing there will be small feature either in darcs or in
Eclipsedarcs - to create compressed output from send command. This
shouldn't be complicated. And would be very helpful for me.

Cheers,
  Radek.

Re: [eclipsedarcs-develop] Push All operation refactoring + Send All Fix.

From: <rad...@gm...> - 2007-03-19 21:22:43

Hi Leif,

> :-) Here's what I did: I run a runtime-workbench with some repo that has a
> few patches. I created a second local repo in a folder. Then I went to the
> project properties page, selected 'Darcs', and added a new repo location, by
> just entering something like 'C:\temp\blarepo'. That gave me the yellow
> warning icon, and the push operation didn't work, so I inferred I had done
> something wrong with the location. I played around a bit, and finally
> entering a URI worked.

Ahh.. This one. I totally forgot about it. It is however flawed and I
will probably remove it.
What I was thinking while implementing this page was to have
repository attached with given project. This is not supported by
RepositoryLocationManager as far as I understand it. Moreover I'm not
sure does it make much sense to have repos for projects only.

What do you think? Should we care and support such a feature or do we
use global repo locations everywhere? If we support it then I think
IDarcsRepository should have link to Projects (if attached to one) and
 RepositoryLocationManager should have the ability to search repos for
given project. Right? I don't think that is very hard to do...

Also, what do we do with pop-up menu for project? I think that we
should either move everything to "Darcs Repository" or back to "Team"
menu. I don't think we should keep it mixed like it is right now for
0.4 release which is coming soon I think. :D Will you take a look at
it?

Cheers!
  Radek.

Re: [eclipsedarcs-develop] Push All operation refactoring + Send All Fix.

From: <rad...@gm...> - 2007-03-20 11:03:32

2007/3/20, Leif Frenzel <lei...@go...>:
> Hi Radek,
>
>
> > What I was thinking while implementing this page was to have
> > repository attached with given project. This is not supported by
> > RepositoryLocationManager as far as I understand it. Moreover I'm not
> > sure does it make much sense to have repos for projects only.
> >
> > What do you think? Should we care and support such a feature or do we
> > use global repo locations everywhere? If we support it then I think
> > IDarcsRepository should have link to Projects (if attached to one) and
> > RepositoryLocationManager should have the ability to search repos for
> > given project. Right? I don't think that is very hard to do...
> I think having repos globally simplifies things.
>
> Also, I think to associate them is very easy. Every project has a repository
> provider attached to it, if it is shared with Darcs. From that, it can be
> determined to which repo a project belongs (that is for instance what the
> [Right-click project] > Darcs repo > Browse action does. It checks with
> which repo the project is shared, and then opens that repo in the Darcs
> Browsing perspective (or creates it there, if it is not yet there). That's a
> feature I use quite often, btw. ;-)

Yes, but this is one (project) to one (repo) mapping (Correct me if
I'm talking rubbish). What I was thinking was one (project) to many
(repos) mapping.
Let's say I change my code, and from time to time I push to my own
branch. Now I want to create patch against (or push to or whatever)
main repository - I would have to have two repositories attached to
one project.

But anyway, I agree to use global list. It is not an issue for me.

> > Also, what do we do with pop-up menu for project? I think that we
> > should either move everything to "Darcs Repository" or back to "Team"
> > menu. I don't think we should keep it mixed like it is right now for
> > 0.4 release which is coming soon I think. :D Will you take a look at
> > it?
> Yes, I'll try it out again. My suspicion is, however, that there is a bug in
> the menu handling in Eclipse. I have found it not possible to have submenus
> under 'Team'. If that is actually so, there might be a chance that it will
> be fixed in Eclipse 3.3, however.

Great!


  Cheers,
  Radek.