You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by sv...@mobsol.be on 2006/10/18 23:27:04 UTC
[PATCH]: in the Windows build system, don't create the serf project if openssl isn't available.
Hi,
attached is a patch for the windows build system (gen-win.py) that disables
creation of the serf project and all its properties when the openssl package
isn't available.
OpenSSL is a required package for serf, but it's optional for the rest of the
targets so it makes sense to not have it installed.
regards,
Lieven.
[[[
In the Windows buildsystem, don't create the serf project if openssl can't be
found.
* build/generator/gen_win.py
(__init__): extracted check whether serf is available in a separate
function.
(_find_serf): test if both serf and openssl directories are available
(get_install_targets, get_external_project, get_win_defines,
get_win_includes, get_win_lib_dirs, get_win_libs,
write_neon_project_file, write_serf_project_file): instead of only
checking for serf, also check for openssl.
]]]
----------------------------------------------------------------
This message was sent using IMP, the Internet Messaging Program.
Re: [PATCH]: in the Windows build system, don't create the serf project if openssl isn't available.
Posted by Ivan Zhakov <ch...@gmail.com>.
On 10/18/06, svnlgo@mobsol.be <sv...@mobsol.be> wrote:
> Updated patch based on the remarks of zhakov, who happens to be sitting next to
> me.
>
> Lieven.
>
> Quoting svnlgo@mobsol.be:
>
> > Hi,
> >
> > attached is a patch for the windows build system (gen-win.py) that disables
> > creation of the serf project and all its properties when the openssl package
> > isn't available.
> >
> > OpenSSL is a required package for serf, but it's optional for the rest of the
> > targets so it makes sense to not have it installed.
> >
> > regards,
> >
> > Lieven.
>
> [[[
> In the Windows buildsystem, don't create the serf project if openssl can't be
> found.
>
> * build/generator/gen_win.py
> (__init__): extracted check whether serf is available in a separate
> function.
> (_find_serf): test if both serf and openssl directories are available
> (get_install_targets, get_external_project, get_win_defines,
> get_win_includes, get_win_lib_dirs, get_win_libs,
> write_neon_project_file, write_serf_project_file): instead of only
> checking for serf, also check for openssl.
> ]]]
>
>
Looks good for me, +1 for commit.
--
Ivan Zhakov
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Re: [PATCH]: in the Windows build system, don't create the serf project if openssl isn't available.
Posted by sv...@mobsol.be.
Updated patch based on the remarks of zhakov, who happens to be sitting next to
me.
Lieven.
Quoting svnlgo@mobsol.be:
> Hi,
>
> attached is a patch for the windows build system (gen-win.py) that disables
> creation of the serf project and all its properties when the openssl package
> isn't available.
>
> OpenSSL is a required package for serf, but it's optional for the rest of the
> targets so it makes sense to not have it installed.
>
> regards,
>
> Lieven.
[[[
In the Windows buildsystem, don't create the serf project if openssl can't be
found.
* build/generator/gen_win.py
(__init__): extracted check whether serf is available in a separate
function.
(_find_serf): test if both serf and openssl directories are available
(get_install_targets, get_external_project, get_win_defines,
get_win_includes, get_win_lib_dirs, get_win_libs,
write_neon_project_file, write_serf_project_file): instead of only
checking for serf, also check for openssl.
]]]
----------------------------------------------------------------
This message was sent using IMP, the Internet Messaging Program.