You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@apr.apache.org by Daniel Sahlberg <da...@gmail.com> on 2021/07/21 12:18:30 UTC
svn:ignore a few more paths to help TortoiseSVN's build system
Hi,
TortoiseSVN is importing APR and APR-util via an svn:externals definition:
[[[
https://svn.apache.org/repos/asf/apr/apr/tags/1.6.5 apr
https://svn.apache.org/repos/asf/apr/apr-util/tags/1.6.1 apr-util
]]]
The build system creates a few new directories within the apr and apr-util
directories for temporary build files. These clutter svn status and makes
it more difficult to see if there are any real changes.
I would propose to add a few more folders in svn:ignore. (I have already
done similar modifications in the Subversion repository, see r1891003).
[[[
Index: apr
===================================================================
--- apr (revision 1891700)
+++ apr (working copy)
Property changes on: apr
___________________________________________________________________
Modified: svn:ignore
## -14,8 +14,16 ##
LibNTR
Debug
DebugNT
+debug_win32
+debug_win32_static
+debug_x64
+debug_x64_static
Release
ReleaseNT
+release_win32
+release_win32_static
+release_x64
+release_x64_static
*.ncb
*.opt
*.plg
Index: apr-util
===================================================================
--- apr-util (revision 1891700)
+++ apr-util (working copy)
Property changes on: apr-util
___________________________________________________________________
Modified: svn:ignore
## -17,7 +17,15 ##
apu-*-config
apu-config.out
Debug
+debug_win32
+debug_win32_static
+debug_x64
+debug_x64_static
Release
+release_win32
+release_win32_static
+release_x64
+release_x64_static
LibD
LibR
aprutil.opt
]]]
An alternative approach would be to ignore debug_* and release_*, this
might help in case there is a future architecture (eg arm).
(I realise this will be made on /trunk and not affect the tags that
TortoiseSVN is pulling in until there is a new release, but it will be
better in the future).
Kind regards,
Daniel Sahlberg
Re: svn:ignore a few more paths to help TortoiseSVN's build system
Posted by William A Rowe Jr <wr...@rowe-clan.net>.
While I'm a cmake user, there's no reason not to exclude these
directories, they are clearly not in our hierarchy.
On Wed, Jul 21, 2021 at 8:07 AM Eric Covener <co...@gmail.com> wrote:
>
> Seems reasonable to me even on the release branches, will let it stew
> a bit here.
>
> On Wed, Jul 21, 2021 at 8:18 AM Daniel Sahlberg
> <da...@gmail.com> wrote:
> >
> > Hi,
> >
> > TortoiseSVN is importing APR and APR-util via an svn:externals definition:
> > [[[
> > https://svn.apache.org/repos/asf/apr/apr/tags/1.6.5 apr
> > https://svn.apache.org/repos/asf/apr/apr-util/tags/1.6.1 apr-util
> > ]]]
> >
> > The build system creates a few new directories within the apr and apr-util directories for temporary build files. These clutter svn status and makes it more difficult to see if there are any real changes.
> >
> > I would propose to add a few more folders in svn:ignore. (I have already done similar modifications in the Subversion repository, see r1891003).
> >
> > [[[
> > Index: apr
> > ===================================================================
> > --- apr (revision 1891700)
> > +++ apr (working copy)
> >
> > Property changes on: apr
> > ___________________________________________________________________
> > Modified: svn:ignore
> > ## -14,8 +14,16 ##
> > LibNTR
> > Debug
> > DebugNT
> > +debug_win32
> > +debug_win32_static
> > +debug_x64
> > +debug_x64_static
> > Release
> > ReleaseNT
> > +release_win32
> > +release_win32_static
> > +release_x64
> > +release_x64_static
> > *.ncb
> > *.opt
> > *.plg
> > Index: apr-util
> > ===================================================================
> > --- apr-util (revision 1891700)
> > +++ apr-util (working copy)
> >
> > Property changes on: apr-util
> > ___________________________________________________________________
> > Modified: svn:ignore
> > ## -17,7 +17,15 ##
> > apu-*-config
> > apu-config.out
> > Debug
> > +debug_win32
> > +debug_win32_static
> > +debug_x64
> > +debug_x64_static
> > Release
> > +release_win32
> > +release_win32_static
> > +release_x64
> > +release_x64_static
> > LibD
> > LibR
> > aprutil.opt
> > ]]]
> >
> > An alternative approach would be to ignore debug_* and release_*, this might help in case there is a future architecture (eg arm).
> >
> > (I realise this will be made on /trunk and not affect the tags that TortoiseSVN is pulling in until there is a new release, but it will be better in the future).
> >
> > Kind regards,
> > Daniel Sahlberg
> >
>
>
> --
> Eric Covener
> covener@gmail.com
Re: svn:ignore a few more paths to help TortoiseSVN's build system
Posted by Eric Covener <co...@gmail.com>.
Seems reasonable to me even on the release branches, will let it stew
a bit here.
On Wed, Jul 21, 2021 at 8:18 AM Daniel Sahlberg
<da...@gmail.com> wrote:
>
> Hi,
>
> TortoiseSVN is importing APR and APR-util via an svn:externals definition:
> [[[
> https://svn.apache.org/repos/asf/apr/apr/tags/1.6.5 apr
> https://svn.apache.org/repos/asf/apr/apr-util/tags/1.6.1 apr-util
> ]]]
>
> The build system creates a few new directories within the apr and apr-util directories for temporary build files. These clutter svn status and makes it more difficult to see if there are any real changes.
>
> I would propose to add a few more folders in svn:ignore. (I have already done similar modifications in the Subversion repository, see r1891003).
>
> [[[
> Index: apr
> ===================================================================
> --- apr (revision 1891700)
> +++ apr (working copy)
>
> Property changes on: apr
> ___________________________________________________________________
> Modified: svn:ignore
> ## -14,8 +14,16 ##
> LibNTR
> Debug
> DebugNT
> +debug_win32
> +debug_win32_static
> +debug_x64
> +debug_x64_static
> Release
> ReleaseNT
> +release_win32
> +release_win32_static
> +release_x64
> +release_x64_static
> *.ncb
> *.opt
> *.plg
> Index: apr-util
> ===================================================================
> --- apr-util (revision 1891700)
> +++ apr-util (working copy)
>
> Property changes on: apr-util
> ___________________________________________________________________
> Modified: svn:ignore
> ## -17,7 +17,15 ##
> apu-*-config
> apu-config.out
> Debug
> +debug_win32
> +debug_win32_static
> +debug_x64
> +debug_x64_static
> Release
> +release_win32
> +release_win32_static
> +release_x64
> +release_x64_static
> LibD
> LibR
> aprutil.opt
> ]]]
>
> An alternative approach would be to ignore debug_* and release_*, this might help in case there is a future architecture (eg arm).
>
> (I realise this will be made on /trunk and not affect the tags that TortoiseSVN is pulling in until there is a new release, but it will be better in the future).
>
> Kind regards,
> Daniel Sahlberg
>
--
Eric Covener
covener@gmail.com