Patchwork [pull,request] Pull request for branch lttng-libust-quick-fix

login
register
mail settings
Submitter Thomas Petazzoni
Date March 26, 2012, 10:33 p.m.
Message ID <cover.1332801197.git.thomas.petazzoni@free-electrons.com>
Download mbox
Permalink /patch/148839/
State Not Applicable
Headers show

Pull-request

git://git.free-electrons.com/users/thomas-petazzoni/buildroot.git lttng-libust-quick-fix

Comments

Thomas Petazzoni - March 26, 2012, 10:33 p.m.
The following changes since commit b5fd0f2709cab72f48da12a5804dd8670fffbf6e:

  Change the /etc/mtab symlink to use an absolute path (2012-03-19 17:08:13 +0100)

are available in the git repository at:
  git://git.free-electrons.com/users/thomas-petazzoni/buildroot.git lttng-libust-quick-fix

Thomas Petazzoni (1):
      lttng-libust: rename patch so that it is taken into account

 ... lttng-libust-1.9.2-fix-overflow-32-bits.patch} |    0
 1 files changed, 0 insertions(+), 0 deletions(-)
 rename package/lttng-libust/{lttng-libust-fix-overflow-32-bits.patch => lttng-libust-1.9.2-fix-overflow-32-bits.patch} (100%)

Thanks,
Thomas Petazzoni - March 26, 2012, 10:33 p.m.
The package/lttng-libust/ contains a patch prefixed with
lttng-libust-1.9.2, so the other patch simply prefixed with
lttng-libust does not get applied. Rename the second patch properly.

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 ... lttng-libust-1.9.2-fix-overflow-32-bits.patch} |    0
 1 files changed, 0 insertions(+), 0 deletions(-)
 rename package/lttng-libust/{lttng-libust-fix-overflow-32-bits.patch => lttng-libust-1.9.2-fix-overflow-32-bits.patch} (100%)

diff --git a/package/lttng-libust/lttng-libust-fix-overflow-32-bits.patch b/package/lttng-libust/lttng-libust-1.9.2-fix-overflow-32-bits.patch
similarity index 100%
rename from package/lttng-libust/lttng-libust-fix-overflow-32-bits.patch
rename to package/lttng-libust/lttng-libust-1.9.2-fix-overflow-32-bits.patch
Arnout Vandecappelle - March 28, 2012, 8:26 p.m.
On Tuesday 27 March 2012 00:33:21 Thomas Petazzoni wrote:
> The package/lttng-libust/ contains a patch prefixed with
> lttng-libust-1.9.2, so the other patch simply prefixed with
> lttng-libust does not get applied. Rename the second patch properly.

 I thought we agreed to avoid versioned patches as much as possible, no?
So this should rather be a rename of 
lttng-libust-1.9.2-dont-build-cxx-tests-when-not-available.patch


 Regards,
 Arnout
Peter Korsgaard - April 4, 2012, 8:07 p.m.
>>>>> "Arnout" == Arnout Vandecappelle <arnout@mind.be> writes:

 Arnout> On Tuesday 27 March 2012 00:33:21 Thomas Petazzoni wrote:
 >> The package/lttng-libust/ contains a patch prefixed with
 >> lttng-libust-1.9.2, so the other patch simply prefixed with
 >> lttng-libust does not get applied. Rename the second patch properly.

 Arnout>  I thought we agreed to avoid versioned patches as much as possible, no?
 Arnout> So this should rather be a rename of 
 Arnout> lttng-libust-1.9.2-dont-build-cxx-tests-when-not-available.patch

I agree. I'll rename it.