Skip to content

Tweaks on project. #10

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

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open

Tweaks on project. #10

wants to merge 16 commits into from

Conversation

0rangeFox
Copy link

A lot has been changed, as you can see. Namespaces and other improvements have been implemented. I don't know what else to say, but otherwise you can check it out and if you have any questions just let me know. And thank you for creating this project.

@philipperemy
Copy link
Owner

@0rangeFox how do you build the project? Can you paste the commands like make install and so on?

@0rangeFox
Copy link
Author

In this case, this project is not for building, but for incorporating into already developed projects.

@0rangeFox
Copy link
Author

But if you want to make this so that it is "buildable", we would have to make it as a library in CMakeLists. And then officially make it available in MinGW as a package to make it easier to implement this project in their projects.

@philipperemy
Copy link
Owner

@0rangeFox okay because I'm not really familiar with C++ I haven't done that in years. How do we compile the binaries?

@0rangeFox
Copy link
Author

@philipperemy
Copy link
Owner

philipperemy commented May 1, 2021

@0rangeFox

We need to add:

set (CMAKE_CXX_STANDARD 11)
cd easy-encryption
cmake -B build
cmake --build build

That creates a .a lib.

How do you do once you have a .a lib? Sorry my C++ is very limited.

I am on MacOS. Can you just pate a few commands that I can run to be able to run the wrapper?

python wrapper.py

@0rangeFox
Copy link
Author

As you can see, in the latest commits I have already added this, and it already has the workflows working, you can go to my repository and get the latest package that was compiled on macOS and so on to test it on your PC. And another thing, don't merge this pull request yet, it's necessary to solve the Vigenere encryption first, it's giving some problems.

image

@0rangeFox 0rangeFox closed this May 11, 2022
@0rangeFox 0rangeFox reopened this Jun 8, 2024
@0rangeFox
Copy link
Author

Hey again 👋! After a good few years, we're back again with this issue 😉. So here's the thing, I've updated the code and now it works perfectly. I hope you like the changes that have been made. And thanks for your project!

@philipperemy
Copy link
Owner

Thank you will have a look soon :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants