I agree with Stuart. We're not Debian. We don't subpackage by default. If we 
do, there needs to be a good reason.


On December 28, 2018 8:04:51 PM GMT+01:00, Stuart Henderson 
<[email protected]> wrote:
>On 2018/12/28 19:30, Rafael Sadowski wrote:
>> On Fri Dec 28, 2018 at 02:31:49PM +0000, Stuart Henderson wrote:
>> > Why subpackage this?
>> 
>> Because nobody needs boost-build to run/build any ports. Let's be
>> unfair, nobody needs boost-build except *maybe* a handful user.
>
>If it was something huge then it would make sense. But really I don't
>see the point in splitting it, it's got to be pretty small, I haven't
>looked at the size of the new files but I can't imagine the whole thing
>can be more than a MB or so can it? Less if the two identical files
>b2 and bjam were hardlinked together.
>
>> Doas another subpackge hurts?
>
>Yes, it's extra scaffolding and extra risk of getting things wrong
>both in the port itself, and in ports which might end up needing it
>in the future.
>
>I'm not going to insist but I really don't see a good reason to split
>it. But if you are going to split it anyway see comments inline.;
>
>> > -- 
>> > Sent from a phone, apologies for poor formatting.
>> > 
>> > On 28 December 2018 13:41:22 Rafael Sadowski
><[email protected]> wrote:
>> > 
>> > > Hi all!
>> > > 
>> > > Our boost.build executables b2 and bjam are useless without the
>*.jam
>> > > stuff. The diff below want to fix the situation withe a third
>> > > sub-package named "build".
>> > > 
>> > > do-install bits from NetBSD and DESCR from FreeBSD.
>> > > Noticed by Ivan Krylov via openbsd@freenode. Thanks!
>> > > 
>> > > Comments, OK?
>> > > 
>> > > Rafael Sadowski
>> > > 
>> 
>> New diff with a compileall.py call at the end of the do-install step,
>> after feedback from semarie@ and aja@ Thanks!
>> 
>> Index: Makefile
>> ===================================================================
>> RCS file: /cvs/ports/devel/boost/Makefile,v
>> retrieving revision 1.79
>> diff -u -p -u -p -r1.79 Makefile
>> --- Makefile 24 Dec 2018 23:34:27 -0000      1.79
>> +++ Makefile 28 Dec 2018 18:18:16 -0000
>> @@ -2,6 +2,7 @@
>>  
>>  COMMENT-main=       free peer-reviewed portable C++ source libraries
>>  COMMENT-md= machine-dependent libraries for boost
>> +COMMENT-build=      extensible cross-platform build tool suite
>>  
>>  # XXX if updating, try to remove
>-D_LIBCPP_ENABLE_CXX17_REMOVED_AUTO_PTR
>>  # XXX in archivers/innoextract
>> @@ -9,7 +10,8 @@ VERSION=    1.66.0
>>  DISTNAME=   boost_${VERSION:S/./_/g}
>>  PKGNAME-main=       boost-${VERSION}
>>  PKGNAME-md= boost-md-${VERSION}
>> -REVISION-main=      1
>> +PKGNAME-build=      boost-build-${VERSION}
>> +REVISION-main=      2
>>  CATEGORIES= devel
>>  MASTER_SITES=       ${MASTER_SITE_SOURCEFORGE:=boost/}
>>  EXTRACT_SUFX=       .tar.bz2
>> @@ -76,7 +78,7 @@ WANTLIB += ${COMPILER_LIBCXX} bz2 c icud
>>  
>>  COMPILER= base-clang ports-gcc
>>  
>> -MULTI_PACKAGES=     -main -md
>> +MULTI_PACKAGES=     -main -md -build
>>  
>>  MODULES=    lang/python
>>  MODPY_RUNDEP=       No
>> @@ -167,12 +169,22 @@ do-build:
>>      ./b2 ${BJAM_CONFIG} python=${MODPY_DEFAULT_VERSION_3}\
>>  
>>  do-install:
>> -    ${INSTALL_PROGRAM} ${WRKSRC}/tools/build/src/engine/bin.*/{b2,bjam}
>\
>> -            ${PREFIX}/bin
>>      ${INSTALL_DATA} ${WRKSRC}/stage/lib/lib!(*.so) ${PREFIX}/lib
>>      @cd ${WRKSRC} && \
>>              find boost -type d -exec ${INSTALL_DATA_DIR} 
>> ${PREFIX}/include/{}
>\;
>>      @cd ${WRKSRC} && \
>>              find boost ! -name \*.orig -type f -exec ${INSTALL_DATA} {}
>${PREFIX}/include/{} \;
>> +    # -build
>> +    ${INSTALL_PROGRAM} ${WRKSRC}/tools/build/src/engine/bin.*/{b2,bjam}
>\
>> +            ${PREFIX}/bin
>> +    ${INSTALL_DATA_DIR} ${PREFIX}/share/boost-build
>> +    @cd ${WRKSRC}/tools/build/src && \
>> +            pax -r -w -p pm -s ':^./engine.*$$::' \
>> +            -s ':^.*\.orig$$::' . ${PREFIX}/share/boost-build
>> +    ${INSTALL_DATA_DIR} ${PREFIX}/share/examples/boost-build
>> +    ${INSTALL_DATA} ${WRKSRC}/tools/build/example/site-config.jam \
>> +            ${PREFIX}/share/examples/boost-build
>> +    ${MODPY_BIN} ${MODPY_LIBDIR}/compileall.py \
>> +            ${PREFIX}/share/boost-build/
>>  
>>  .include <bsd.port.mk>
>> Index: pkg/DESCR-build
>> ===================================================================
>> RCS file: pkg/DESCR-build
>> diff -N pkg/DESCR-build
>> --- /dev/null        1 Jan 1970 00:00:00 -0000
>> +++ pkg/DESCR-build  28 Dec 2018 18:18:16 -0000
>> @@ -0,0 +1,7 @@
>> +Boost.Build is an easy way to build C++ projects, everywhere. You
>name you
>> +executables and libraries and list their sources.
>
>"You name *your* executables"...
>
>> +
>> +Boost.Build takes care about compiling your sources with right
>options,
>
>"with the right options"
>
>> +creating static and shared libraries, making executables, and other
>chores --
>> +whether you're using gcc, msvc, or a dozen more supported C++
>compilers -- on
>> +Windows, OSX, Linux and commercial UNIX systems.
>> Index: pkg/PLIST-build
>> ===================================================================
>> RCS file: pkg/PLIST-build
>> diff -N pkg/PLIST-build
>> --- /dev/null        1 Jan 1970 00:00:00 -0000
>> +++ pkg/PLIST-build  28 Dec 2018 18:18:16 -0000
>> @@ -0,0 +1,362 @@
>> +@comment $OpenBSD: PLIST-build,v$
>> +@bin bin/b2
>> +@bin bin/bjam
>
>missing @conflict marker.
>
>> +share/boost-build/
>> +share/boost-build/__init__.py
>> +share/boost-build/__init__.pyc
>> +share/boost-build/bootstrap.jam
>> +share/boost-build/build/
>> +share/boost-build/build-system.jam
>> +share/boost-build/build/__init__.py
>> +share/boost-build/build/__init__.pyc
>> +share/boost-build/build/ac.jam
>> +share/boost-build/build/alias.jam
>> +share/boost-build/build/alias.py
>> +share/boost-build/build/alias.pyc
>> +share/boost-build/build/build-request.jam
>> +share/boost-build/build/build_request.py
>> +share/boost-build/build/build_request.pyc
>> +share/boost-build/build/config-cache.jam
>> +share/boost-build/build/configure.jam
>> +share/boost-build/build/configure.py
>> +share/boost-build/build/configure.pyc
>> +share/boost-build/build/engine.py
>> +share/boost-build/build/engine.pyc
>> +share/boost-build/build/errors.py
>> +share/boost-build/build/errors.pyc
>> +share/boost-build/build/feature.jam
>> +share/boost-build/build/feature.py
>> +share/boost-build/build/feature.pyc
>> +share/boost-build/build/generators.jam
>> +share/boost-build/build/generators.py
>> +share/boost-build/build/generators.pyc
>> +share/boost-build/build/project.jam
>> +share/boost-build/build/project.py
>> +share/boost-build/build/project.pyc
>> +share/boost-build/build/property-set.jam
>> +share/boost-build/build/property.jam
>> +share/boost-build/build/property.py
>> +share/boost-build/build/property.pyc
>> +share/boost-build/build/property_set.py
>> +share/boost-build/build/property_set.pyc
>> +share/boost-build/build/readme.txt
>> +share/boost-build/build/scanner.jam
>> +share/boost-build/build/scanner.py
>> +share/boost-build/build/scanner.pyc
>> +share/boost-build/build/targets.jam
>> +share/boost-build/build/targets.py
>> +share/boost-build/build/targets.pyc
>> +share/boost-build/build/toolset.jam
>> +share/boost-build/build/toolset.py
>> +share/boost-build/build/toolset.pyc
>> +share/boost-build/build/type.jam
>> +share/boost-build/build/type.py
>> +share/boost-build/build/type.pyc
>> +share/boost-build/build/version.jam
>> +share/boost-build/build/version.py
>> +share/boost-build/build/version.pyc
>> +share/boost-build/build/virtual-target.jam
>> +share/boost-build/build/virtual_target.py
>> +share/boost-build/build/virtual_target.pyc
>> +share/boost-build/build_system.py
>> +share/boost-build/build_system.pyc
>> +share/boost-build/contrib/
>> +share/boost-build/contrib/__init__.py
>> +share/boost-build/contrib/__init__.pyc
>> +share/boost-build/contrib/boost.jam
>> +share/boost-build/contrib/boost.py
>> +share/boost-build/contrib/boost.pyc
>> +share/boost-build/contrib/modular.jam
>> +share/boost-build/contrib/tntnet.jam
>> +share/boost-build/contrib/wxFormBuilder.jam
>> +share/boost-build/exceptions.py
>> +share/boost-build/exceptions.pyc
>> +share/boost-build/kernel/
>> +share/boost-build/kernel/boost-build.jam
>> +share/boost-build/kernel/bootstrap.jam
>> +share/boost-build/kernel/bootstrap.py
>> +share/boost-build/kernel/bootstrap.pyc
>> +share/boost-build/kernel/class.jam
>> +share/boost-build/kernel/errors.jam
>> +share/boost-build/kernel/modules.jam
>> +share/boost-build/manager.py
>> +share/boost-build/manager.pyc
>> +share/boost-build/options/
>> +share/boost-build/options/help.jam
>> +share/boost-build/tools/
>> +share/boost-build/tools/__init__.py
>> +share/boost-build/tools/__init__.pyc
>> +share/boost-build/tools/acc.jam
>> +share/boost-build/tools/asciidoctor.jam
>> +share/boost-build/tools/auto-index.jam
>> +share/boost-build/tools/bison.jam
>> +share/boost-build/tools/boostbook-config.jam
>> +share/boost-build/tools/boostbook.jam
>> +share/boost-build/tools/borland.jam
>> +share/boost-build/tools/builtin.jam
>> +share/boost-build/tools/builtin.py
>> +share/boost-build/tools/builtin.pyc
>> +share/boost-build/tools/bzip2.jam
>> +share/boost-build/tools/cast.jam
>> +share/boost-build/tools/cast.py
>> +share/boost-build/tools/cast.pyc
>> +share/boost-build/tools/clang-darwin.jam
>> +share/boost-build/tools/clang-linux.jam
>> +share/boost-build/tools/clang-vxworks.jam
>> +share/boost-build/tools/clang-win.jam
>> +share/boost-build/tools/clang.jam
>> +share/boost-build/tools/common.jam
>> +share/boost-build/tools/common.py
>> +share/boost-build/tools/common.pyc
>> +share/boost-build/tools/common_clang_vc.jam
>> +share/boost-build/tools/como-linux.jam
>> +share/boost-build/tools/como-win.jam
>> +share/boost-build/tools/como.jam
>> +share/boost-build/tools/convert.jam
>> +share/boost-build/tools/cray.jam
>> +share/boost-build/tools/cw-config.jam
>> +share/boost-build/tools/cw.jam
>> +share/boost-build/tools/cygwin.jam
>> +share/boost-build/tools/darwin.jam
>> +share/boost-build/tools/darwin.py
>> +share/boost-build/tools/darwin.pyc
>> +share/boost-build/tools/diab.jam
>> +share/boost-build/tools/dmc.jam
>> +share/boost-build/tools/docutils.jam
>> +share/boost-build/tools/doxproc.py
>> +share/boost-build/tools/doxproc.pyc
>> +share/boost-build/tools/doxygen/
>> +share/boost-build/tools/doxygen-config.jam
>> +share/boost-build/tools/doxygen.jam
>> +share/boost-build/tools/doxygen/windows-paths-check.doxyfile
>> +share/boost-build/tools/doxygen/windows-paths-check.hpp
>> +share/boost-build/tools/emscripten.jam
>> +share/boost-build/tools/features/
>> +share/boost-build/tools/features/__init_features__.jam
>> +share/boost-build/tools/features/address-model-feature.jam
>> +share/boost-build/tools/features/allow-feature.jam
>> +share/boost-build/tools/features/architecture-feature.jam
>> +share/boost-build/tools/features/archiveflags-feature.jam
>> +share/boost-build/tools/features/asmflags-feature.jam
>> +share/boost-build/tools/features/build-feature.jam
>> +share/boost-build/tools/features/cflags-feature.jam
>> +share/boost-build/tools/features/conditional-feature.jam
>> +share/boost-build/tools/features/cxx-template-depth-feature.jam
>> +share/boost-build/tools/features/cxxabi-feature.jam
>> +share/boost-build/tools/features/cxxflags-feature.jam
>> +share/boost-build/tools/features/cxxstd-feature.jam
>> +share/boost-build/tools/features/debug-feature.jam
>> +share/boost-build/tools/features/define-feature.jam
>> +share/boost-build/tools/features/dependency-feature.jam
>> +share/boost-build/tools/features/dll-feature.jam
>> +share/boost-build/tools/features/exception-feature.jam
>> +share/boost-build/tools/features/fflags-feature.jam
>> +share/boost-build/tools/features/file-feature.jam
>> +share/boost-build/tools/features/find-lib-feature.jam
>> +share/boost-build/tools/features/flags-feature.jam
>> +share/boost-build/tools/features/include-feature.jam
>> +share/boost-build/tools/features/instruction-set-feature.jam
>> +share/boost-build/tools/features/internal-feature.jam
>> +share/boost-build/tools/features/library-feature.jam
>> +share/boost-build/tools/features/link-feature.jam
>> +share/boost-build/tools/features/linkflags-feature.jam
>> +share/boost-build/tools/features/location-feature.jam
>> +share/boost-build/tools/features/location-prefix-feature.jam
>> +share/boost-build/tools/features/name-feature.jam
>> +share/boost-build/tools/features/objcflags-feature.jam
>> +share/boost-build/tools/features/optimization-feature.jam
>> +share/boost-build/tools/features/os-feature.jam
>> +share/boost-build/tools/features/rtti-feature.jam
>> +share/boost-build/tools/features/runtime-feature.jam
>> +share/boost-build/tools/features/search-feature.jam
>> +share/boost-build/tools/features/source-feature.jam
>> +share/boost-build/tools/features/stdlib-feature.jam
>> +share/boost-build/tools/features/strip-feature.jam
>> +share/boost-build/tools/features/tag-feature.jam
>> +share/boost-build/tools/features/threadapi-feature.jam
>> +share/boost-build/tools/features/threading-feature.jam
>> +share/boost-build/tools/features/toolset-feature.jam
>> +share/boost-build/tools/features/user-interface-feature.jam
>> +share/boost-build/tools/features/variant-feature.jam
>> +share/boost-build/tools/features/version-feature.jam
>> +share/boost-build/tools/features/warnings-feature.jam
>> +share/boost-build/tools/fop.jam
>> +share/boost-build/tools/fortran.jam
>> +share/boost-build/tools/gcc.jam
>> +share/boost-build/tools/gcc.py
>> +share/boost-build/tools/gcc.pyc
>> +share/boost-build/tools/generate.jam
>> +share/boost-build/tools/generators/
>> +share/boost-build/tools/generators/__init_generators__.jam
>> +share/boost-build/tools/generators/archive-generator.jam
>> +share/boost-build/tools/generators/c-compiling-generator.jam
>> +share/boost-build/tools/generators/dummy-generator.jam
>> +share/boost-build/tools/generators/lib-generator.jam
>> +share/boost-build/tools/generators/linking-generator.jam
>> +share/boost-build/tools/generators/prebuilt-lib-generator.jam
>> +share/boost-build/tools/generators/searched-lib-generator.jam
>> +share/boost-build/tools/gettext.jam
>> +share/boost-build/tools/gfortran.jam
>> +share/boost-build/tools/hp_cxx.jam
>> +share/boost-build/tools/hpfortran.jam
>> +share/boost-build/tools/ifort.jam
>> +share/boost-build/tools/intel-darwin.jam
>> +share/boost-build/tools/intel-linux.jam
>> +share/boost-build/tools/intel-vxworks.jam
>> +share/boost-build/tools/intel-win.jam
>> +share/boost-build/tools/intel.jam
>> +share/boost-build/tools/lex.jam
>> +share/boost-build/tools/libjpeg.jam
>> +share/boost-build/tools/libpng.jam
>> +share/boost-build/tools/libtiff.jam
>> +share/boost-build/tools/link.jam
>> +share/boost-build/tools/lzma.jam
>> +share/boost-build/tools/make.jam
>> +share/boost-build/tools/make.py
>> +share/boost-build/tools/make.pyc
>> +share/boost-build/tools/mc.jam
>> +share/boost-build/tools/mc.py
>> +share/boost-build/tools/mc.pyc
>> +share/boost-build/tools/message.jam
>> +share/boost-build/tools/message.py
>> +share/boost-build/tools/message.pyc
>> +share/boost-build/tools/midl.jam
>> +share/boost-build/tools/midl.py
>> +share/boost-build/tools/midl.pyc
>> +share/boost-build/tools/mipspro.jam
>> +share/boost-build/tools/mpi.jam
>> +share/boost-build/tools/msvc-config.jam
>> +share/boost-build/tools/msvc.jam
>> +share/boost-build/tools/msvc.py
>> +share/boost-build/tools/msvc.pyc
>> +share/boost-build/tools/notfile.jam
>> +share/boost-build/tools/notfile.py
>> +share/boost-build/tools/notfile.pyc
>> +share/boost-build/tools/package.jam
>> +share/boost-build/tools/package.py
>> +share/boost-build/tools/package.pyc
>> +share/boost-build/tools/pathscale.jam
>> +share/boost-build/tools/pch.jam
>> +share/boost-build/tools/pch.py
>> +share/boost-build/tools/pch.pyc
>> +share/boost-build/tools/pgi.jam
>> +share/boost-build/tools/python-config.jam
>> +share/boost-build/tools/python.jam
>> +share/boost-build/tools/qcc.jam
>> +share/boost-build/tools/qt.jam
>> +share/boost-build/tools/qt3.jam
>> +share/boost-build/tools/qt4.jam
>> +share/boost-build/tools/qt5.jam
>> +share/boost-build/tools/quickbook-config.jam
>> +share/boost-build/tools/quickbook.jam
>> +share/boost-build/tools/rc.jam
>> +share/boost-build/tools/rc.py
>> +share/boost-build/tools/rc.pyc
>> +share/boost-build/tools/sass.jam
>> +share/boost-build/tools/stage.jam
>> +share/boost-build/tools/stage.py
>> +share/boost-build/tools/stage.pyc
>> +share/boost-build/tools/stlport.jam
>> +share/boost-build/tools/sun.jam
>> +share/boost-build/tools/symlink.jam
>> +share/boost-build/tools/symlink.py
>> +share/boost-build/tools/symlink.pyc
>> +share/boost-build/tools/testing-aux.jam
>> +share/boost-build/tools/testing.jam
>> +share/boost-build/tools/testing.py
>> +share/boost-build/tools/testing.pyc
>> +share/boost-build/tools/types/
>> +share/boost-build/tools/types/__init__.py
>> +share/boost-build/tools/types/__init__.pyc
>> +share/boost-build/tools/types/adoc.jam
>> +share/boost-build/tools/types/asm.jam
>> +share/boost-build/tools/types/asm.py
>> +share/boost-build/tools/types/asm.pyc
>> +share/boost-build/tools/types/cpp.jam
>> +share/boost-build/tools/types/cpp.py
>> +share/boost-build/tools/types/cpp.pyc
>> +share/boost-build/tools/types/css.jam
>> +share/boost-build/tools/types/docbook.jam
>> +share/boost-build/tools/types/exe.jam
>> +share/boost-build/tools/types/exe.py
>> +share/boost-build/tools/types/exe.pyc
>> +share/boost-build/tools/types/html.jam
>> +share/boost-build/tools/types/html.py
>> +share/boost-build/tools/types/html.pyc
>> +share/boost-build/tools/types/lib.jam
>> +share/boost-build/tools/types/lib.py
>> +share/boost-build/tools/types/lib.pyc
>> +share/boost-build/tools/types/man.jam
>> +share/boost-build/tools/types/markdown.jam
>> +share/boost-build/tools/types/markdown.py
>> +share/boost-build/tools/types/markdown.pyc
>> +share/boost-build/tools/types/obj.jam
>> +share/boost-build/tools/types/obj.py
>> +share/boost-build/tools/types/obj.pyc
>> +share/boost-build/tools/types/objc.jam
>> +share/boost-build/tools/types/pdf.jam
>> +share/boost-build/tools/types/preprocessed.jam
>> +share/boost-build/tools/types/preprocessed.py
>> +share/boost-build/tools/types/preprocessed.pyc
>> +share/boost-build/tools/types/qt.jam
>> +share/boost-build/tools/types/register.jam
>> +share/boost-build/tools/types/rsp.jam
>> +share/boost-build/tools/types/rsp.py
>> +share/boost-build/tools/types/rsp.pyc
>> +share/boost-build/tools/types/sass-type.jam
>> +share/boost-build/tools/types/xml.jam
>> +share/boost-build/tools/unix.jam
>> +share/boost-build/tools/unix.py
>> +share/boost-build/tools/unix.pyc
>> +share/boost-build/tools/vacpp.jam
>> +share/boost-build/tools/vmsdecc.jam
>> +share/boost-build/tools/whale.jam
>> +share/boost-build/tools/xlcpp.jam
>> +share/boost-build/tools/xlf.jam
>> +share/boost-build/tools/xsltproc/
>> +share/boost-build/tools/xsltproc-config.jam
>> +share/boost-build/tools/xsltproc.jam
>> +share/boost-build/tools/xsltproc/included.xsl
>> +share/boost-build/tools/xsltproc/test.xml
>> +share/boost-build/tools/xsltproc/test.xsl
>> +share/boost-build/tools/zlib.jam
>> +share/boost-build/util/
>> +share/boost-build/util/__init__.py
>> +share/boost-build/util/__init__.pyc
>> +share/boost-build/util/assert.jam
>> +share/boost-build/util/container.jam
>> +share/boost-build/util/doc.jam
>> +share/boost-build/util/indirect.jam
>> +share/boost-build/util/indirect.py
>> +share/boost-build/util/indirect.pyc
>> +share/boost-build/util/logger.py
>> +share/boost-build/util/logger.pyc
>> +share/boost-build/util/numbers.jam
>> +share/boost-build/util/option.jam
>> +share/boost-build/util/option.py
>> +share/boost-build/util/option.pyc
>> +share/boost-build/util/order.jam
>> +share/boost-build/util/order.py
>> +share/boost-build/util/order.pyc
>> +share/boost-build/util/os.jam
>> +share/boost-build/util/os_j.py
>> +share/boost-build/util/os_j.pyc
>> +share/boost-build/util/path.jam
>> +share/boost-build/util/path.py
>> +share/boost-build/util/path.pyc
>> +share/boost-build/util/print.jam
>> +share/boost-build/util/regex.jam
>> +share/boost-build/util/regex.py
>> +share/boost-build/util/regex.pyc
>> +share/boost-build/util/sequence.jam
>> +share/boost-build/util/sequence.py
>> +share/boost-build/util/sequence.pyc
>> +share/boost-build/util/set.jam
>> +share/boost-build/util/set.py
>> +share/boost-build/util/set.pyc
>> +share/boost-build/util/string.jam
>> +share/boost-build/util/utility.jam
>> +share/boost-build/util/utility.py
>> +share/boost-build/util/utility.pyc
>> +share/examples/boost-build/
>> +share/examples/boost-build/site-config.jam

-- 
Antoine

Reply via email to