Bug 232109 - deskutils/treesheets: Update to 1.0.1
Summary: deskutils/treesheets: Update to 1.0.1
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Steve Wills
URL: https://github.com/aardappel/treeshee...
Keywords: easy, needs-qa
Depends on:
Blocks:
 
Reported: 2018-10-09 04:37 UTC by lightside
Modified: 2018-10-09 13:53 UTC (History)
1 user (show)

See Also:
lightside: maintainer-feedback+


Attachments
Proposed patch (since 475857 revision) (1.97 KB, patch)
2018-10-09 04:37 UTC, lightside
no flags Details | Diff
Proposed patch (since 475857 revision) (1.98 KB, patch)
2018-10-09 04:44 UTC, lightside
lightside: maintainer-approval+
Details | Diff
Proposed patch (since 475857 revision) (2.09 KB, patch)
2018-10-09 05:31 UTC, lightside
lightside: maintainer-approval+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description lightside 2018-10-09 04:37:16 UTC
Created attachment 197954 [details]
Proposed patch (since 475857 revision)

Patch to update deskutils/treesheets port from 1.0.0 to 1.0.1 version.

Look following link for changes:
https://github.com/aardappel/treesheets/compare/v1.0.0...v1.0.1

- Use C++17 compatible compiler for build [1]
- Adapt value for PORTDATA variable to install "scripts" and "translations" directories
- Fix build for FreeBSD 10, with using appropriate libc++ include files

Reference links:
1. Now always requiring C++17
https://github.com/aardappel/treesheets/commit/2b657dc5a8ca645d11dfcb5f64a114e5d53c26d1

The build was tested on FreeBSD 10.4 amd64.
Comment 1 lightside 2018-10-09 04:44:41 UTC
Created attachment 197955 [details]
Proposed patch (since 475857 revision)
Comment 2 Kubilay Kocak freebsd_committer freebsd_triage 2018-10-09 04:48:24 UTC
The following block probably needs a comment with explanation of the need for it, if nothing else than to help the reviewer/committer grok/evaluate the pattern:

+.if (${OPSYS} == FreeBSD && ${OSVERSION} < 1100000) || make(makesum)
Comment 3 lightside 2018-10-09 05:31:58 UTC
Created attachment 197956 [details]
Proposed patch (since 475857 revision)

(In reply to Kubilay Kocak from comment #2)
> The following block probably needs a comment with explanation of the need
> for it, <..>
> +.if (${OPSYS} == FreeBSD && ${OSVERSION} < 1100000) || make(makesum)
Ok, I added some comment for related block.

Some examples of related errors, if not using this:
-8<--
% make
<..>
In file included from lobster_impl.cpp:2:
In file included from ../lobster/src/lobster/stdafx.h:42:
In file included from /usr/include/c++/v1/unordered_map:350:
/usr/include/c++/v1/__hash_table:1165:43: error: conflicting types for '__hash_table<_Tp, _Hash, _Equal, _Alloc>'
__hash_table<_Tp, _Hash, _Equal, _Alloc>::__hash_table()
                                          ^
/usr/include/c++/v1/__hash_table:852:5: note: previous declaration is here
    __hash_table()
    ^
/usr/include/c++/v1/__hash_table:1233:43: error: conflicting types for '__hash_table<_Tp, _Hash, _Equal, _Alloc>'
__hash_table<_Tp, _Hash, _Equal, _Alloc>::__hash_table(__hash_table&& __u)
                                          ^
/usr/include/c++/v1/__hash_table:866:5: note: previous declaration is here
    __hash_table(__hash_table&& __u)
    ^
<..>
In file included from lobster_impl.cpp:2:
In file included from ../lobster/src/lobster/stdafx.h:84:
../lobster/src/lobster/tools.h:714:25: error: unknown type name 'string_view'; did you mean 'std::experimental::string_view'?
inline string operator+(string_view a, string_view b) {
                        ^~~~~~~~~~~
                        std::experimental::string_view
/usr/include/c++/v1/experimental/string_view:777:39: note: 'std::experimental::string_view' declared here
  typedef basic_string_view<char>     string_view;
                                      ^
In file included from lobster_impl.cpp:2:
In file included from ../lobster/src/lobster/stdafx.h:84:
../lobster/src/lobster/tools.h:714:40: error: unknown type name 'string_view'; did you mean 'std::experimental::string_view'?
inline string operator+(string_view a, string_view b) {
                                       ^~~~~~~~~~~
                                       std::experimental::string_view
/usr/include/c++/v1/experimental/string_view:777:39: note: 'std::experimental::string_view' declared here
  typedef basic_string_view<char>     string_view;
                                      ^
<..>
% make MAKE_JOBS_UNSAFE=yes
<..>
In file included from main.cpp:2:
./stdafx.h:73:10: fatal error: 'string_view' file not found
#include <string_view>
         ^~~~~~~~~~~~~
1 error generated.
<..>
-->8-
Comment 4 Steve Wills freebsd_committer freebsd_triage 2018-10-09 13:53:00 UTC
Committed, thanks!
Comment 5 commit-hook freebsd_committer freebsd_triage 2018-10-09 13:53:04 UTC
A commit references this bug:

Author: swills
Date: Tue Oct  9 13:52:06 UTC 2018
New revision: 481634
URL: https://svnweb.freebsd.org/changeset/ports/481634

Log:
  deskutils/treesheets: Update to 1.0.1

  PR:		232109
  Submitted by:	lightside <lightside@gmx.com> (maintianer)

Changes:
  head/deskutils/treesheets/Makefile
  head/deskutils/treesheets/distinfo