You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Senthil Kumaran S <se...@collab.net> on 2008/07/08 07:56:38 UTC

[PATCH] Fix buiding in separte directory

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Hi,

I used to build subversion trunk in a separate directory inside the source tree
called 'virtual-neon' with the following command line options:

<snip>
stylesen@stylesen:~/subversion-dev/trunk/virtual-neon$ ../configure
- --prefix=/home/stylesen/subversion-dev/builds/pure-trunk-neon
- --enable-maintainer-mode --with-ssl --with-neon=/usr/local/neon-0.28.2
- --with-apxs=/usr/bin/apxs2 --disable-mod-activation
- --with-jdk=/usr/lib/jvm/java-1.5.0-sun
</snip>

This started breaking with the following error from yesterday (sorry for not
giving the exact revision):

<snip>
/home/stylesen/subversion-dev/trunk/virtual-neon/subversion/libsvn_ra_neon/.libs/libsvn_ra_neon-1.so.0:
undefined reference to `gss_import_name'
/home/stylesen/subversion-dev/trunk/virtual-neon/subversion/libsvn_ra_neon/.libs/libsvn_ra_neon-1.so.0:
undefined reference to `gss_release_name'
/home/stylesen/subversion-dev/trunk/virtual-neon/subversion/libsvn_ra_neon/.libs/libsvn_ra_neon-1.so.0:
undefined reference to `gss_delete_sec_context'
/home/stylesen/subversion-dev/trunk/virtual-neon/subversion/libsvn_ra_neon/.libs/libsvn_ra_neon-1.so.0:
undefined reference to `gss_display_status'
/home/stylesen/subversion-dev/trunk/virtual-neon/subversion/libsvn_ra_neon/.libs/libsvn_ra_neon-1.so.0:
undefined reference to `gss_release_buffer'
/home/stylesen/subversion-dev/trunk/virtual-neon/subversion/libsvn_ra_neon/.libs/libsvn_ra_neon-1.so.0:
undefined reference to `gss_init_sec_context'
/home/stylesen/subversion-dev/trunk/virtual-neon/subversion/libsvn_ra_neon/.libs/libsvn_ra_neon-1.so.0:
undefined reference to `GSS_C_NT_HOSTBASED_SERVICE'
collect2: ld returned 1 exit status
make: *** [subversion/svnlook/svnlook] Error 1
</snip>

The attached patch fixes this for me, where I ve added a dependency of neon for
svnlook, this helps me in building trunk, which was otherwise impossible with
my repeated attempts.

[[[
Fix building in separate directory.

* build.conf
  (svnlook): Add neon dependency for svnlook libs.

Patch by: stylesen
]]]

Thank You.
- --
Senthil Kumaran S
http://www.stylesen.org/
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)

iD8DBQFIcx209o1G+2zNQDgRAsFCAKCZ7S60Ck6dFmm9dxghpvsHxI4bbACeIUu2
wD935JsJO5TDa5kTVy4Ft1E=
=i7nr
-----END PGP SIGNATURE-----

Re: Julian, side effect of r32015?

Posted by Senthil Kumaran S <se...@collab.net>.
Julian Foad wrote:
> I didn't notice this warning, at first:
> 
> subversion/libsvn_subr/cmdline.c:693: no previous prototype for
> 'svn_cmdline__print_xml_prop'
> 
> That's because that file ought to include its header, the new
> "svn_cmdline_private.h".
> 
> Fixing...
> 
> Committed revision 32066.

Oops, my compiler somehow missed it :) Thank You.

-- 
Senthil Kumaran S
http://www.stylesen.org/


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: Julian, side effect of r32015?

Posted by Julian Foad <ju...@btopenworld.com>.
On Thu, 2008-07-10 at 17:15 +0100, Julian Foad wrote:
> On Thu, 2008-07-10 at 21:08 +0530, Senthil Kumaran S wrote:
> > Senthil Kumaran S wrote:
> > > I am attaching the updated patch along with this email.
> > > 
> > > Daniel, I ve incorporated the changes you ve mentioned for the log message.
> > 
> > Updated patch attached after r32060.
> 
> Thanks, Senthil.
> 
> Committed in revision 32065.

I didn't notice this warning, at first:

subversion/libsvn_subr/cmdline.c:693: no previous prototype for
'svn_cmdline__print_xml_prop'

That's because that file ought to include its header, the new
"svn_cmdline_private.h".

Fixing...

Committed revision 32066.

- Julian



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: Julian, side effect of r32015?

Posted by Julian Foad <ju...@btopenworld.com>.
On Thu, 2008-07-10 at 21:08 +0530, Senthil Kumaran S wrote:
> Senthil Kumaran S wrote:
> > I am attaching the updated patch along with this email.
> > 
> > Daniel, I ve incorporated the changes you ve mentioned for the log message.
> 
> Updated patch attached after r32060.

Thanks, Senthil.

Committed in revision 32065.

(The to-be-added file "subversion/include/private/svn_cmdline_private.h"
was missing from this version of the patch but I found it in the
previous version and pasted it in. No problem.)

- Julian


> [[[
> Factor out an XML-printing function that was common to "svn" and "svnlook"
> since r31978.
> 
> This Follows up to r32015, r32018, r32021, r32022, r32043 and the discussions
> on this thread - http://svn.haxx.se/dev/archive-2008-07/0227.shtml



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: Julian, side effect of r32015?

Posted by Senthil Kumaran S <se...@collab.net>.
Senthil Kumaran S wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> 
> Karl Fogel wrote:
>> Of course, the appropriate patch now (after r32043, in which Julian
>> reverted his change) would be somewhat different anyway...
> 
> I am attaching the updated patch along with this email.
> 
> Daniel, I ve incorporated the changes you ve mentioned for the log message.

Updated patch attached after r32060.

[[[
Factor out an XML-printing function that was common to "svn" and "svnlook"
since r31978.

This Follows up to r32015, r32018, r32021, r32022, r32043 and the discussions
on this thread - http://svn.haxx.se/dev/archive-2008-07/0227.shtml

* build.conf
   (libsvn_subr): Add svn_cmdline_private.h to msvc-export.

* subversion/libsvn_subr/cmdline.c
   (): Include svn_xml.h, svn_base64.h
   (svn_cmdline__print_xml_prop): New name for svn_cl__print_xml_prop()

* subversion/svn/props.c
   (): Include svn_cmdline_private.h
   (svn_cl__print_xml_prop): Removed, since it is renamed to
    svn_cmdline__print_xml_prop.
   (svn_cl__print_xml_prop_hash): Use svn_cmdline__print_xml_prop due to
    above change.

* subversion/svn/cl.h
   (svn_cl__print_xml_prop): Move to ..

* subversion/include/private/svn_cmdline_private.h
   (svn_cmdline__print_xml_prop): .. here

* subversion/svn/propget-cmd.c
   (): Include svn_cmdline_private.h
   (print_properties_xml): Use svn_cmdline__print_xml_prop due to above change.
   (svn_cl__propget): Same as above.

* subversion/svnlook/main.c
   (): Include svn_cmdline_private.h
   (print_xml_prop): Removed since we can use svn_cmdline__print_xml_prop now.
   (do_plist): Use svn_cmdline__print_xml_prop due to above changes.

Patch by: stylesen
Suggested by: glasser
]]]

Thank You.
-- 
Senthil Kumaran S
http://www.stylesen.org/


Re: Julian, side effect of r32015?

Posted by Senthil Kumaran S <se...@collab.net>.
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Karl Fogel wrote:
> Of course, the appropriate patch now (after r32043, in which Julian
> reverted his change) would be somewhat different anyway...

I am attaching the updated patch along with this email.

Daniel, I ve incorporated the changes you ve mentioned for the log message.

[[[
Factor out an XML-printing function that was common to "svn" and "svnlook"
since r31978.

This Follows up to r32015, r32018, r32021, r32022, r32043 and the discussions
on this thread - http://svn.haxx.se/dev/archive-2008-07/0227.shtml

* build.conf
  (libsvn_client): Remove svn_client_private.h from msvc-export.
  (libsvn_subr): Add svn_cmdline_private.h to msvc-export.

* subversion/libsvn_subr/cmdline.c
  (): Include svn_xml.h, svn_base64.h
  (svn_cmdline__print_xml_prop): New name for svn_cl__print_xml_prop()

* subversion/svn/props.c
  (): Include svn_cmdline_private.h
  (svn_cl__print_xml_prop): Removed, since it is renamed to
   svn_cmdline__print_xml_prop.
  (svn_cl__print_xml_prop_hash): Use svn_cmdline__print_xml_prop due to
   above change.

* subversion/svn/cl.h
  (svn_cl__print_xml_prop): Move to ..

* subversion/include/private/svn_cmdline_private.h
  (svn_cmdline__print_xml_prop): .. here

* subversion/svn/propget-cmd.c
  (): Include svn_cmdline_private.h
  (print_properties_xml): Use svn_cmdline__print_xml_prop due to above change.
  (svn_cl__propget): Same as above.

* subversion/svnlook/main.c
  (): Include svn_cmdline_private.h
  (print_xml_prop): Removed since we can use svn_cmdline__print_xml_prop now.
  (do_plist): Use svn_cmdline__print_xml_prop due to above changes.

Patch by: stylesen
Suggested by: glasser
]]]

Thank You.
- --
Senthil Kumaran S
http://www.stylesen.org/
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)

iD8DBQFIdcry9o1G+2zNQDgRAn5JAKCgxXQO27sRUJ0rcXrnvPIGeJf4BQCcDP+t
b+RzfeG8vex9hJAUNXS+iBE=
=QVDC
-----END PGP SIGNATURE-----

Re: Julian, side effect of r32015?

Posted by Karl Fogel <kf...@red-bean.com>.
Of course, the appropriate patch now (after r32043, in which Julian
reverted his change) would be somewhat different anyway...

-Karl

Daniel Shahaf <d....@daniel.shahaf.co.il> writes:
> Senthil Kumaran S wrote on Wed, 9 Jul 2008 at 03:11 +0530:
>> Karl Fogel wrote:
>> > Julian, maybe r32015/r32018/r32021 should be redone to put this code in
>> > libsvn_subr and svn_xml_private.h instead?  I know you didn't want to
>> > put property-specific code in the 'svn_xml__' namespace, but that would
>> > appear to be by far the lesser evil here... Thoughts?
>> 
>> How about the attached patch.
>> 
>
> I didn't read the patch, just the log message, but I have a few comments
> nonetheless:
>
>> [[[
>> Follow up to r32015, r32018, r32021, r32022.
>> 
>> Move XML-printing function from libsvn_client to libsvn_subr.
>> 
>
> Why move it?  (I know why; but the log message should say that.)
>
> Also, I think a link to this thread would be useful.
>
>> * build.conf
>>   (svnlook): Remove libsvn_client dependency.
>>   (libsvn_client): Remove svn_client_private.h from msvc-export.
>> 
>> * subversion/libsvn_subr/cmdline.c
>>   (): Include svn_string.h, svn_xml.h, svn_config.h
>>   (svn_cmdline__print_xml_prop): New function.
>> 
>
> It isn't "new".  It is svn_client__print_xml_prop, moved and renamed;
> that's what the log message should say.  It should be clear from the log
> message which symbols are new and which were renamed, and what their old
> and new names are.
>
> e.g.,
>
>    (svn_cmdline__print_xml_prop):  New name for svn_client__print_xml_prop().
>
> You'll also see this form used:
>
>     * foo.c (foo_func):  Move to..
>     * bar.c (bar_func):  ..here.
>
>> * subversion/libsvn_client/xml.c
>>   Removed.
>> 
>
> Please list here the public symbols that existed in this file and what
> was done with them (were they deleted, moved, or renamed, and where to).
>
>> * subversion/svn/props.c
>>   (): Remove svn_client_private.h
>>   (svn_cl__print_xml_prop_hash): Use svn_cmdline__print_xml_prop instead of
>>    svn_client__print_xml_prop.
>> 
>> * subversion/svn/propget-cmd.c
>>   (): Remove svn_client_private.h
>>   (print_properties_xml): Use svn_cmdline__print_xml_prop instead of
>>    svn_client__print_xml_prop.
>>   (svn_cl__propget): Same as above.
>> 
>> * subversion/include/svn_cmdline.h
>>   (svn_cmdline__print_xml_prop): New API.
>> 
>
> It should be declared in include/private/svn_cmdline_private.h.
>
>> * subversion/include/private/svn_client_private.h
>>   Removed.
>> 
>> * subversion/svnlook/main.c
>>   (): Removed svn_client_private.h
>>   (do_plist): Use svn_cmdline__print_xml_prop instead of
>>    svn_client__print_xml_prop.
>> 
>> Patch by: stylesen
>> Suggested by: dglasser
>                 ^
> s/d//
>
>> ]]]
>> 
>> Thank You.
>> 
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
> For additional commands, e-mail: dev-help@subversion.tigris.org

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: Julian, side effect of r32015?

Posted by Daniel Shahaf <d....@daniel.shahaf.co.il>.
Senthil Kumaran S wrote on Wed, 9 Jul 2008 at 03:11 +0530:
> Karl Fogel wrote:
> > Julian, maybe r32015/r32018/r32021 should be redone to put this code in
> > libsvn_subr and svn_xml_private.h instead?  I know you didn't want to
> > put property-specific code in the 'svn_xml__' namespace, but that would
> > appear to be by far the lesser evil here... Thoughts?
> 
> How about the attached patch.
> 

I didn't read the patch, just the log message, but I have a few comments
nonetheless:

> [[[
> Follow up to r32015, r32018, r32021, r32022.
> 
> Move XML-printing function from libsvn_client to libsvn_subr.
> 

Why move it?  (I know why; but the log message should say that.)

Also, I think a link to this thread would be useful.

> * build.conf
>   (svnlook): Remove libsvn_client dependency.
>   (libsvn_client): Remove svn_client_private.h from msvc-export.
> 
> * subversion/libsvn_subr/cmdline.c
>   (): Include svn_string.h, svn_xml.h, svn_config.h
>   (svn_cmdline__print_xml_prop): New function.
> 

It isn't "new".  It is svn_client__print_xml_prop, moved and renamed;
that's what the log message should say.  It should be clear from the log
message which symbols are new and which were renamed, and what their old
and new names are.

e.g.,

   (svn_cmdline__print_xml_prop):  New name for svn_client__print_xml_prop().

You'll also see this form used:

    * foo.c (foo_func):  Move to..
    * bar.c (bar_func):  ..here.

> * subversion/libsvn_client/xml.c
>   Removed.
> 

Please list here the public symbols that existed in this file and what
was done with them (were they deleted, moved, or renamed, and where to).

> * subversion/svn/props.c
>   (): Remove svn_client_private.h
>   (svn_cl__print_xml_prop_hash): Use svn_cmdline__print_xml_prop instead of
>    svn_client__print_xml_prop.
> 
> * subversion/svn/propget-cmd.c
>   (): Remove svn_client_private.h
>   (print_properties_xml): Use svn_cmdline__print_xml_prop instead of
>    svn_client__print_xml_prop.
>   (svn_cl__propget): Same as above.
> 
> * subversion/include/svn_cmdline.h
>   (svn_cmdline__print_xml_prop): New API.
> 

It should be declared in include/private/svn_cmdline_private.h.

> * subversion/include/private/svn_client_private.h
>   Removed.
> 
> * subversion/svnlook/main.c
>   (): Removed svn_client_private.h
>   (do_plist): Use svn_cmdline__print_xml_prop instead of
>    svn_client__print_xml_prop.
> 
> Patch by: stylesen
> Suggested by: dglasser
                ^
s/d//

> ]]]
> 
> Thank You.
> 

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: Julian, side effect of r32015?

Posted by Julian Foad <ju...@btopenworld.com>.
> Karl Fogel wrote:
> > Daniel Shahaf <d....@daniel.shahaf.co.il> writes:
> > [...]
> >> And I agree with him that svnlook shouldn't depend on libsvn_ra.
> > 
> > I suspect you're right.
> > 
> > Julian, maybe r32015/r32018/r32021 should be redone to put this code in
> > libsvn_subr and svn_xml_private.h instead?  I know you didn't want to
> > put property-specific code in the 'svn_xml__' namespace, but that would
> > appear to be by far the lesser evil here... Thoughts?

I am at the BCS/itSMG conference today and tomorrow, so no time to do
more than revert this.

Reverting...

[[[
$ svn merge -c r-32021 -c -32018 -c -32015 .
--- Reverse-merging r32021 into '.':
U    subversion/svn/props.c
U    subversion/svn/propget-cmd.c
U    subversion/libsvn_client/xml.c
U    subversion/svnlook/main.c
--- Reverse-merging r32018 into '.':
U    subversion/include/svn_client.h
D    subversion/include/private/svn_client_private.h
svn: Working copy path 'subversion/include/private/svn_client_private.h' does not exist in repository
]]]

Shucks, what now?

[[[
$ svn revert -R subversion/
Reverted 'subversion/svn/props.c'
Reverted 'subversion/svn/propget-cmd.c'
Reverted 'subversion/include/svn_client.h'
Reverted 'subversion/include/private/svn_client_private.h'
Reverted 'subversion/libsvn_client/xml.c'
Reverted 'subversion/svnlook/main.c'

$ svn merge --ignore-ancestry -c -32021 -c -32018 -c -32015 .
--- Reverse-merging r32021 into '.':
U    subversion/svn/props.c
U    subversion/svn/propget-cmd.c
U    subversion/libsvn_client/xml.c
U    subversion/svnlook/main.c
--- Reverse-merging r32018 into '.':
U    subversion/include/svn_client.h
D    subversion/include/private/svn_client_private.h
--- Reverse-merging r32015 into '.':
U    build.conf
G    subversion/svn/props.c
U    subversion/svn/cl.h
G    subversion/svn/propget-cmd.c
G    subversion/include/svn_client.h
Skipped 'subversion/libsvn_client/xml.c'
G    subversion/svnlook/main.c
]]]

I think "skipped" is where it should delete the file.

[[[
$ svn diff --summarize -c -32021
M      subversion/svn/props.c
M      subversion/svn/propget-cmd.c
M      subversion/libsvn_client/xml.c
M      subversion/svnlook/main.c

$ svn diff --summarize -c -32018
M      subversion/include/svn_client.h
D      subversion/include/private/svn_client_private.h

$ svn diff --summarize -c -32015
M      build.conf
M      subversion/svn/props.c
M      subversion/svn/cl.h
M      subversion/svn/propget-cmd.c
M      subversion/include/svn_client.h
D      subversion/libsvn_client/xml.c
M      subversion/svnlook/main.c
]]]

Yes, confirmed.

[[[
$ svn delete subversion/libsvn_client/xml.c
svn: Use --force to override this restriction
svn: 'subversion/libsvn_client/xml.c' has local modifications

$ svn delete --force subversion/libsvn_client/xml.c
D         subversion/libsvn_client/xml.c
]]]

Build-and-test ... OK up to test 35 of 62.

[[[
$ svn ci
Sending        build.conf
Deleting       subversion/include/private/svn_client_private.h
Deleting       subversion/libsvn_client/xml.c
Sending        subversion/svn/cl.h
Sending        subversion/svn/propget-cmd.c
Sending        subversion/svn/props.c
Sending        subversion/svnlook/main.c
Transmitting file data .....
Committed revision 32043.
]]]


Senthil Kumaran S wrote:
> How about the attached patch.
> 
> [[[
> Follow up to r32015, r32018, r32021, r32022.
> 
> Move XML-printing function from libsvn_client to libsvn_subr.
> [...]

Sure, that might be good. If someone would like to check it over and
commit it that would be great.

- Julian



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: Julian, side effect of r32015?

Posted by Senthil Kumaran S <se...@collab.net>.
Karl Fogel wrote:
> Daniel Shahaf <d....@daniel.shahaf.co.il> writes:
>> I think this is the side effect that David predicted here:
>>
>> http://thread.gmane.org/gmane.comp.version-control.subversion.svn/28621/focus=102169
>>
>> And I agree with him that svnlook shouldn't depend on libsvn_ra.
> 
> I suspect you're right.
> 
> Julian, maybe r32015/r32018/r32021 should be redone to put this code in
> libsvn_subr and svn_xml_private.h instead?  I know you didn't want to
> put property-specific code in the 'svn_xml__' namespace, but that would
> appear to be by far the lesser evil here... Thoughts?

How about the attached patch.

[[[
Follow up to r32015, r32018, r32021, r32022.

Move XML-printing function from libsvn_client to libsvn_subr.

* build.conf
   (svnlook): Remove libsvn_client dependency.
   (libsvn_client): Remove svn_client_private.h from msvc-export.

* subversion/libsvn_subr/cmdline.c
   (): Include svn_string.h, svn_xml.h, svn_config.h
   (svn_cmdline__print_xml_prop): New function.

* subversion/svn/props.c
   (): Remove svn_client_private.h
   (svn_cl__print_xml_prop_hash): Use svn_cmdline__print_xml_prop instead of
    svn_client__print_xml_prop.

* subversion/svn/propget-cmd.c
   (): Remove svn_client_private.h
   (print_properties_xml): Use svn_cmdline__print_xml_prop instead of
    svn_client__print_xml_prop.
   (svn_cl__propget): Same as above.

* subversion/include/svn_cmdline.h
   (svn_cmdline__print_xml_prop): New API.

* subversion/include/private/svn_client_private.h
   Removed.

* subversion/svnlook/main.c
   (): Removed svn_client_private.h
   (do_plist): Use svn_cmdline__print_xml_prop instead of
    svn_client__print_xml_prop.

* subversion/libsvn_client/xml.c
   Removed.

Patch by: stylesen
Suggested by: dglasser
]]]

Thank You.
-- 
Senthil Kumaran S
http://www.stylesen.org/


Julian, side effect of r32015? (was: [PATCH] Fix buiding in separte directory)

Posted by Karl Fogel <kf...@red-bean.com>.
Daniel Shahaf <d....@daniel.shahaf.co.il> writes:
> I think this is the side effect that David predicted here:
>
> http://thread.gmane.org/gmane.comp.version-control.subversion.svn/28621/focus=102169
>
> And I agree with him that svnlook shouldn't depend on libsvn_ra.

I suspect you're right.

Julian, maybe r32015/r32018/r32021 should be redone to put this code in
libsvn_subr and svn_xml_private.h instead?  I know you didn't want to
put property-specific code in the 'svn_xml__' namespace, but that would
appear to be by far the lesser evil here... Thoughts?

-Karl

> Daniel
>
> Senthil Kumaran S wrote on Tue, 8 Jul 2008 at 13:26 +0530:
>> -----BEGIN PGP SIGNED MESSAGE-----
>> Hash: SHA1
>> 
>> Hi,
>> 
>> I used to build subversion trunk in a separate directory inside the source tree
>> called 'virtual-neon' with the following command line options:
>> 
>> <snip>
>> stylesen@stylesen:~/subversion-dev/trunk/virtual-neon$ ../configure
>> - --prefix=/home/stylesen/subversion-dev/builds/pure-trunk-neon
>> - --enable-maintainer-mode --with-ssl --with-neon=/usr/local/neon-0.28.2
>> - --with-apxs=/usr/bin/apxs2 --disable-mod-activation
>> - --with-jdk=/usr/lib/jvm/java-1.5.0-sun
>> </snip>
>> 
>> This started breaking with the following error from yesterday (sorry for not
>> giving the exact revision):
>> 
>> <snip>
>> /home/stylesen/subversion-dev/trunk/virtual-neon/subversion/libsvn_ra_neon/.libs/libsvn_ra_neon-1.so.0:
>> undefined reference to `gss_import_name'
>> /home/stylesen/subversion-dev/trunk/virtual-neon/subversion/libsvn_ra_neon/.libs/libsvn_ra_neon-1.so.0:
>> undefined reference to `gss_release_name'
>> /home/stylesen/subversion-dev/trunk/virtual-neon/subversion/libsvn_ra_neon/.libs/libsvn_ra_neon-1.so.0:
>> undefined reference to `gss_delete_sec_context'
>> /home/stylesen/subversion-dev/trunk/virtual-neon/subversion/libsvn_ra_neon/.libs/libsvn_ra_neon-1.so.0:
>> undefined reference to `gss_display_status'
>> /home/stylesen/subversion-dev/trunk/virtual-neon/subversion/libsvn_ra_neon/.libs/libsvn_ra_neon-1.so.0:
>> undefined reference to `gss_release_buffer'
>> /home/stylesen/subversion-dev/trunk/virtual-neon/subversion/libsvn_ra_neon/.libs/libsvn_ra_neon-1.so.0:
>> undefined reference to `gss_init_sec_context'
>> /home/stylesen/subversion-dev/trunk/virtual-neon/subversion/libsvn_ra_neon/.libs/libsvn_ra_neon-1.so.0:
>> undefined reference to `GSS_C_NT_HOSTBASED_SERVICE'
>> collect2: ld returned 1 exit status
>> make: *** [subversion/svnlook/svnlook] Error 1
>> </snip>
>> 
>> The attached patch fixes this for me, where I ve added a dependency of neon for
>> svnlook, this helps me in building trunk, which was otherwise impossible with
>> my repeated attempts.
>> 
>> [[[
>> Fix building in separate directory.
>> 
>> * build.conf
>>   (svnlook): Add neon dependency for svnlook libs.
>> 
>> Patch by: stylesen
>> ]]]
>> 
>> Thank You.
>> - --
>> Senthil Kumaran S
>> http://www.stylesen.org/
>> -----BEGIN PGP SIGNATURE-----
>> Version: GnuPG v1.4.6 (GNU/Linux)
>> 
>> iD8DBQFIcx209o1G+2zNQDgRAsFCAKCZ7S60Ck6dFmm9dxghpvsHxI4bbACeIUu2
>> wD935JsJO5TDa5kTVy4Ft1E=
>> =i7nr
>> -----END PGP SIGNATURE-----
>> 
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
> For additional commands, e-mail: dev-help@subversion.tigris.org

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: [PATCH] Fix buiding in separte directory

Posted by Daniel Shahaf <d....@daniel.shahaf.co.il>.
I think this is the side effect that David predicted here:

http://thread.gmane.org/gmane.comp.version-control.subversion.svn/28621/focus=102169

And I agree with him that svnlook shouldn't depend on libsvn_ra.

Daniel

Senthil Kumaran S wrote on Tue, 8 Jul 2008 at 13:26 +0530:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> 
> Hi,
> 
> I used to build subversion trunk in a separate directory inside the source tree
> called 'virtual-neon' with the following command line options:
> 
> <snip>
> stylesen@stylesen:~/subversion-dev/trunk/virtual-neon$ ../configure
> - --prefix=/home/stylesen/subversion-dev/builds/pure-trunk-neon
> - --enable-maintainer-mode --with-ssl --with-neon=/usr/local/neon-0.28.2
> - --with-apxs=/usr/bin/apxs2 --disable-mod-activation
> - --with-jdk=/usr/lib/jvm/java-1.5.0-sun
> </snip>
> 
> This started breaking with the following error from yesterday (sorry for not
> giving the exact revision):
> 
> <snip>
> /home/stylesen/subversion-dev/trunk/virtual-neon/subversion/libsvn_ra_neon/.libs/libsvn_ra_neon-1.so.0:
> undefined reference to `gss_import_name'
> /home/stylesen/subversion-dev/trunk/virtual-neon/subversion/libsvn_ra_neon/.libs/libsvn_ra_neon-1.so.0:
> undefined reference to `gss_release_name'
> /home/stylesen/subversion-dev/trunk/virtual-neon/subversion/libsvn_ra_neon/.libs/libsvn_ra_neon-1.so.0:
> undefined reference to `gss_delete_sec_context'
> /home/stylesen/subversion-dev/trunk/virtual-neon/subversion/libsvn_ra_neon/.libs/libsvn_ra_neon-1.so.0:
> undefined reference to `gss_display_status'
> /home/stylesen/subversion-dev/trunk/virtual-neon/subversion/libsvn_ra_neon/.libs/libsvn_ra_neon-1.so.0:
> undefined reference to `gss_release_buffer'
> /home/stylesen/subversion-dev/trunk/virtual-neon/subversion/libsvn_ra_neon/.libs/libsvn_ra_neon-1.so.0:
> undefined reference to `gss_init_sec_context'
> /home/stylesen/subversion-dev/trunk/virtual-neon/subversion/libsvn_ra_neon/.libs/libsvn_ra_neon-1.so.0:
> undefined reference to `GSS_C_NT_HOSTBASED_SERVICE'
> collect2: ld returned 1 exit status
> make: *** [subversion/svnlook/svnlook] Error 1
> </snip>
> 
> The attached patch fixes this for me, where I ve added a dependency of neon for
> svnlook, this helps me in building trunk, which was otherwise impossible with
> my repeated attempts.
> 
> [[[
> Fix building in separate directory.
> 
> * build.conf
>   (svnlook): Add neon dependency for svnlook libs.
> 
> Patch by: stylesen
> ]]]
> 
> Thank You.
> - --
> Senthil Kumaran S
> http://www.stylesen.org/
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.6 (GNU/Linux)
> 
> iD8DBQFIcx209o1G+2zNQDgRAsFCAKCZ7S60Ck6dFmm9dxghpvsHxI4bbACeIUu2
> wD935JsJO5TDa5kTVy4Ft1E=
> =i7nr
> -----END PGP SIGNATURE-----
> 

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org