mbox

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

Message ID cover.1332801197.git.thomas.petazzoni@free-electrons.com
State Not Applicable
Headers show

Pull-request

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

Message

Thomas Petazzoni March 26, 2012, 10:33 p.m. UTC 
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,

Comments

Thomas Petazzoni March 26, 2012, 10:33 p.m. UTC | #1 
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. UTC | #2 
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. UTC | #3 
>>>>> "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.