-
Notifications
You must be signed in to change notification settings - Fork 74
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Replace APR libraries with std:: libraries (abandoned) #74
Comments
After the capnp, I'd like to set for another advanture and bring c++17 (update CI, MSVC for Win,..) and then we can drop lots of dependencies |
Ok, I will start in on this. |
Or you can clone numenta/nupic.core (1st level clone) so you can make PRs to both easily |
That sounds dangerous to me. I don't want to accidentally change anything
in numenta/nupic.core. And since I don't understand GIT very well I should
keep it simple.
…On Wed, Sep 19, 2018 at 6:48 AM breznak ***@***.***> wrote:
But first I want to start over with a fresh clone of
htm-community/nupic.cpp
Or you can clone numenta/nupic.core (1st level clone) so you can make PRs
to both easily
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
<#74 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AFBa_8CmlEiCOp8-tfQ0BhNMuSZ98Lrfks5uckspgaJpZM4WUNOb>
.
|
Ok, cloned htm-community/nupic.cpp
New branch for this project is "replace_APR"
…On Wed, Sep 19, 2018 at 6:54 AM David E Keeney ***@***.***> wrote:
That sounds dangerous to me. I don't want to accidentally change anything
in numenta/nupic.core. And since I don't understand GIT very well I should
keep it simple.
On Wed, Sep 19, 2018 at 6:48 AM breznak ***@***.***> wrote:
> But first I want to start over with a fresh clone of
> htm-community/nupic.cpp
>
> Or you can clone numenta/nupic.core (1st level clone) so you can make PRs
> to both easily
>
> —
> You are receiving this because you were assigned.
> Reply to this email directly, view it on GitHub
> <#74 (comment)>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AFBa_8CmlEiCOp8-tfQ0BhNMuSZ98Lrfks5uckspgaJpZM4WUNOb>
> .
>
|
don't worry, wouldn't happen. One thing is to clone something, other are push privileges. I haven't found the branch, nor clone of project, in your profile yet. So when you're ready, please open a PR. |
This PR is abandoned. I was taking this in too large of chunks. And in the process of trying to get MinGW to work, ended up with massive white-space changes that were mostly Just line-ending changes but it made compares almost impossible. So, See Roadmap Issue #81 for how I propose we proceed. |
Closing, reopen if needed |
Another placeholder for things to do.
Delete the APR library dependency and see what breaks. Fix it by replacing it with calls to std:: libraries.
In the OS folder the Directories and Path objects can be implemented with #include filesystem . This of course requires switching to C++17.
Edit: part of #47
The text was updated successfully, but these errors were encountered: