[chrono] #13557 Boost 1.67.0 chrono library Windows build installs NTFS junction

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

[chrono] #13557 Boost 1.67.0 chrono library Windows build installs NTFS junction

Boost - Dev mailing list
Hi Boost.Chrono devs,

I have some additional information about
https://svn.boost.org/trac10/ticket/13557 . (Also tracked at
https://github.com/StephanTLavavej/mingw-distro/issues/58 , it affects
both MSVC and MinGW.)

As Joshua Morrison discovered, copying
libs/chrono/stopwatches/include/boost/chrono/. to boost/chrono before
building is a workaround that prevents the junction and hardlink from
being created.

Also, I mentioned this on Twitter and René Rivera said: "Those links
are supposed to appear only for the git checkout of @Boost_Libraries
and are created on demand when headers are reference during the header
scanning. Creating those for a release is a bug in the release
process." (https://twitter.com/grafikrobot/status/1011077139511369728
)

I'm still not sure what the correct fix is, but it seems that
boost/chrono should contain the stopwatches files, if they're being
referenced by other headers.

Thanks,
STL

_______________________________________________
Unsubscribe & other changes: http://lists.boost.org/mailman/listinfo.cgi/boost
Reply | Threaded
Open this post in threaded view
|

Re: [chrono] #13557 Boost 1.67.0 chrono library Windows build installs NTFS junction

Boost - Dev mailing list
AMDG

On 06/28/2018 12:39 PM, Stephan T. Lavavej via Boost wrote:

> Hi Boost.Chrono devs,
>
> I have some additional information about
> https://svn.boost.org/trac10/ticket/13557 . (Also tracked at
> https://github.com/StephanTLavavej/mingw-distro/issues/58 , it affects
> both MSVC and MinGW.)
>
> As Joshua Morrison discovered, copying
> libs/chrono/stopwatches/include/boost/chrono/. to boost/chrono before
> building is a workaround that prevents the junction and hardlink from
> being created.
>
> Also, I mentioned this on Twitter and René Rivera said: "Those links
> are supposed to appear only for the git checkout of @Boost_Libraries
> and are created on demand when headers are reference during the header
> scanning. Creating those for a release is a bug in the release
> process." (https://twitter.com/grafikrobot/status/1011077139511369728
> )
>

  I agree with René.  The files are supposed to be
moved as part of building the release.

The script that creates the combined boost/ directory is here:
https://github.com/boostorg/release-tools/blob/develop/MakeBoostDistro.py

> I'm still not sure what the correct fix is, but it seems that
> boost/chrono should contain the stopwatches files, if they're being
> referenced by other headers.
>

In Christ,
Steven Watanabe

_______________________________________________
Unsubscribe & other changes: http://lists.boost.org/mailman/listinfo.cgi/boost
Reply | Threaded
Open this post in threaded view
|

Re: [chrono] #13557 Boost 1.67.0 chrono library Windows build installs NTFS junction

Boost - Dev mailing list
In reply to this post by Boost - Dev mailing list
AMDG

On 06/28/2018 12:39 PM, Stephan T. Lavavej via Boost wrote:
> <snip>
>
> I'm still not sure what the correct fix is, but it seems that
> boost/chrono should contain the stopwatches files, if they're being
> referenced by other headers.
>

I think this will fix it, but I haven't tested it:
https://github.com/boostorg/release-tools/pull/12

In Christ,
Steven Watanabe

_______________________________________________
Unsubscribe & other changes: http://lists.boost.org/mailman/listinfo.cgi/boost