Patch reverted!
Maxim.
On 11/23/2015 14:20, Riku Voipio wrote:
On 20 November 2015 at 21:39, Maxim Uvarov <maxim.uva...@linaro.org> wrote:
CC Riku if he is not in the list.
On 11/20/2015 22:16, Mike Holmes wrote:
I checked with Anders, my summary of the case.
Riku intended this patch would allow ODP to build on a plain vanilla build
server and that goal is not met by this patch.
My understanding was that he wanted to just prepare tarball with sources
from odp.git. There was not intend to build
that sources to binaries.
But to make it happen we need to pass ./configure which tests for different
dependencies. I.e. without ./configure
there is no make dist target with generates .tar.gz.
Agree that this patch has to be reverted due it does not fix exact problem.
It does not fix the exact problem, but it still might be useful on
it's own. Still, I don't mind if this patch is reverted.
Anders, Mike, thanks!
Maxim.
On 20 November 2015 at 13:29, Anders Roxell <anders.rox...@linaro.org
<mailto:anders.rox...@linaro.org>> wrote:
On 20 November 2015 at 19:23, Maxim Uvarov
<maxim.uva...@linaro.org <mailto:maxim.uva...@linaro.org>> wrote:
> On 11/20/2015 20:07, Anders Roxell wrote:
>>
>> On 20 November 2015 at 15:45, Mike Holmes
<mike.hol...@linaro.org <mailto:mike.hol...@linaro.org>> wrote:
>>>
>>> Do we have a review for this ?
>>
>> No because it didn't work due to we always depends on libssl-dev
>> (debian) or openssl-devel (fedora).
>>
>> This is the output if we have a buildserver that doesn't have any
>> pre-installed packages except for
>> the standard, gcc, make, autotools:
>> checking for joinable pthread attribute... PTHREAD_CREATE_JOINABLE
>> checking if more special flags are required for pthreads... no
>> checking for PTHREAD_PRIO_INHERIT... yes
>> checking for EVP_EncryptInit in -lcrypto... no
>> configure: error: in `/odp':
>> configure: error: OpenSSL libraries required
>> See `config.log' for more details
>>
>>
>> So it makes more sense to update the CI build servers to do the
correct
>> thing.
>>
>> That's why I think this patch should be reverted.
>>
>> Cheers,
>> Anders
>
> I tested that and it generated correct tar.gz.
We both agree that this patch generates a tar.gz file.
> What is relation between that
> patch and libssl-dev?
The changelog states that you can generate the source package without
installing the build-deps right, and libssl-dev is that a build
dependency?
I would say yes!
> Patch just adds "source" argument to build source. It does not
modify any
> dependencies.
> In my case ./builddeb source works, and without source it's the same
> behaviour as original.
> I think you log is not related to that patch.
And my log shows what happens if you _NOT_ install any build
dependencies that
is stated in the control file.
Cheers,
Anders
>
> Maxim.
>
>
>
>
>>
>>> On 20 November 2015 at 09:39, Maxim Uvarov
<maxim.uva...@linaro.org <mailto:maxim.uva...@linaro.org>>
>>> wrote:
>>>>
>>>> Merged,
>>>> Maxim.
>>>>
>>>>
>>>> On 11/19/2015 09:35, Riku Voipio wrote:
>>>>>
>>>>> For CI loop, we have separate job to build binaries
(build-package).
>>>>> To make it possible to build source package without first
installing
>>>>> all build-deps, add a new optional parameter "source" to build
>>>>> only source package.
>>>>>
>>>>> Signed-off-by: Riku Voipio <riku.voi...@linaro.org
<mailto:riku.voi...@linaro.org>>
>>>>> ---
>>>>> scripts/builddeb | 7 ++++++-
>>>>> 1 file changed, 6 insertions(+), 1 deletion(-)
>>>>>
>>>>> diff --git a/scripts/builddeb b/scripts/builddeb
>>>>> index 829fd1d..28927ea 100755
>>>>> --- a/scripts/builddeb
>>>>> +++ b/scripts/builddeb
>>>>> @@ -23,6 +23,11 @@ if [ $(egrep "\.([a-z0-9]{8}\>|dirty)"
.scmversion
>>>>> |wc
>>>>> -l) -gt 0 ]; then
>>>>> dch --newversion ${version}-1 --urgency low "not a
official
>>>>> release!"
>>>>> fi
>>>>> -debuild -us -uc
>>>>> +if [ "$1" = source ]
>>>>> +then
>>>>> + dpkg-buildpackage -S -d
>>>>> +else
>>>>> + debuild -us -uc
>>>>> +fi
>>>>> popd
>>>>> popd
>>>>
>>>>
>>>> _______________________________________________
>>>> lng-odp mailing list
>>>> lng-odp@lists.linaro.org <mailto:lng-odp@lists.linaro.org>
>>>> https://lists.linaro.org/mailman/listinfo/lng-odp
>>>
>>>
>>>
>>>
>>> --
>>> Mike Holmes
>>> Technical Manager - Linaro Networking Group
>>> Linaro.org │ Open source software for ARM SoCs
>>>
>>>
>>>
>>> _______________________________________________
>>> lng-odp mailing list
>>> lng-odp@lists.linaro.org <mailto:lng-odp@lists.linaro.org>
>>> https://lists.linaro.org/mailman/listinfo/lng-odp
>>>
>
--
Mike Holmes
Technical Manager - Linaro Networking Group
Linaro.org <http://www.linaro.org/>***│ *Open source software for ARM SoCs
_______________________________________________
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp
