Mirror of the lttng-dev mailing list
 help / color / mirror / Atom feed
From: Daniel.Thibault@drdc-rddc.gc.ca (Thibault, Daniel)
Subject: [lttng-dev] [PATCH] [RESUBMISSION] lttng-tools lib/lttng-ctl/lttng-ctl.c : Document return values and more
Date: Mon, 30 Jan 2012 13:52:05 -0500	[thread overview]
Message-ID: <3CA1A65C03C1A34C81BBC6272782E6968089CF@valcartierex01.valcartier.drdc-rddc.gc.ca> (raw)
In-Reply-To: <4F26DD60.2090909@polymtl.ca>

-----Message d'origine-----
De?: David Goulet [mailto:david.goulet at polymtl.ca] 
Envoy??: 30 janvier 2012 13:12

> I'm having problem applying those patches. It fails at the third one and I think
> it's because you change lttng-ctl.c in the second patch and the third one is not
> based on those changes.

   But it is!?

> Can you please send me three separate emails for those patches (one per mail)
> and had your "Signed-off-by" also (just add "-s" to git commit, it will do it
> automatically for you). It helps me a lot for review and merge.

   Er...I don't seem to have the option of adding "-s", 'cause I'm using EGit from Eclipse.  The only options I get when producing a patch are a) output to clipboard or file, b) export in git patch format or not, and c) the number of lines of context.  Never mind, found it in Window: Preferences: Team: Git: Commit dialog.

   Now, out here we have a separate problem getting at the git repositories using the git: protocol.  The port isn't open in out firewall.  A request has been submitted to get that opened.  Meantime, I must download a local copy of the git commits and import them as regular projects before "gitifying" them.  Bottom line: it works but there are extra hoops to jump through for now.  Once our firewall problem is solved, it'll be a lot easier for me to follow the code's evolution.

   So, in order to fix the problem you're having with the "third patch" ([PATCH] lttng-tools lib/lttng-ctl/lttng-ctl.c : Various code fixes), I'll download the current version (I see I'm 15 commits behind) and regenerate patches from there.

Daniel U. Thibault
R & D pour la d?fense Canada - Valcartier (RDDC Valcartier) / Defence R&D Canada - Valcartier (DRDC Valcartier)
Syst?me de syst?mes (SdS) / System of Systems (SoS)
Solutions informatiques et exp?rimentations (SIE) / Computing Solutions and Experimentations (CSE)
2459 Boul. Pie XI Nord
Qu?bec, QC  G3J 1X5
CANADA
Vox : (418) 844-4000 x4245
Fax : (418) 844-4538
NAC: 918V QSDJ
Gouvernement du Canada / Government of Canada
<http://www.valcartier.drdc-rddc.gc.ca/>



  reply	other threads:[~2012-01-30 18:52 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <mailman.1.1327683602.12686.lttng-dev@lists.lttng.org>
2012-01-27 18:41 ` [lttng-dev] [PATCH] " Thibault, Daniel
2012-01-27 19:46   ` David Goulet
2012-01-27 20:49     ` [lttng-dev] [PATCH] [RESUBMISSION] " Thibault, Daniel
2012-01-30 18:11       ` David Goulet
2012-01-30 18:52         ` Thibault, Daniel [this message]
2012-01-30 20:27           ` Alexandre Montplaisir
2012-01-30 21:12             ` [lttng-dev] git: vs http: Thibault, Daniel
2012-01-30 20:27         ` [lttng-dev] [PATCH] lttng-tools Catching up on miscellaneous string and comment fixes (from 5c73c59) Thibault, Daniel
2012-01-30 21:02           ` David Goulet
2012-01-30 20:31         ` [lttng-dev] [PATCH] lttng-tools add_context.c : Fixing memory leaks Thibault, Daniel
2012-01-30 21:18           ` David Goulet
2012-01-30 21:39             ` Thibault, Daniel
2012-01-30 21:44               ` David Goulet
2012-01-30 21:58             ` Thibault, Daniel
2012-01-30 22:05               ` David Goulet
2012-01-30 20:58         ` [lttng-dev] [PATCH] lttng-tools lttng-ctl, calibrate and create : Document return values Thibault, Daniel
2012-01-30 21:04         ` [lttng-dev] [PATCH] lttng-tools : create.c : Simplify create_session() Thibault, Daniel
2012-01-30 21:26         ` [lttng-dev] [PATCH] Rewrites lttng-ctl's set_session_daemon_path(), fix snprintf() return value tests so the code works with both GNU C < 2.1 and >= 2.1 Thibault, Daniel
2012-01-30 21:35           ` David Goulet

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=3CA1A65C03C1A34C81BBC6272782E6968089CF@valcartierex01.valcartier.drdc-rddc.gc.ca \
    --to=daniel.thibault@drdc-rddc.gc.ca \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox