Skip to content
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

8 docker build #12

Merged
merged 1 commit into from
May 27, 2020
Merged

8 docker build #12

merged 1 commit into from
May 27, 2020

Conversation

millergarym
Copy link
Contributor

There are still some issues with this docker build of GreatSPN.
Recreating this PR to help track them.

@amparore amparore merged commit e21d3ae into greatspn:master May 27, 2020
@amparore
Copy link
Collaborator

I moved the added files to the docker/fedora directory

@millergarym
Copy link
Contributor Author

@amparore
Thanks for merging the PR.
With the move the docker_build.sh needs to be modified.

current

docker build -t gspn . -f fedora.dockerfile

The build context needs to be a the top level.
Maybe something like

docker build -t gspn ../.. -f fedora.dockerfile
                                         ^
                                         might need path here

@amparore
Copy link
Collaborator

amparore commented May 28, 2020 via email

@millergarym
Copy link
Contributor Author

millergarym commented May 31, 2020

Isn't it better to get the sources from the repository?

Getting the source from the repository during the docker build couples git and docker together.
I feel this is bad practise, it makes the build overly complex (eg download.sh) and make local development more difficult.
I am recently new to Petri Nets and from what I can see GreatSPN is one of the best and easiest editors out there, but I haven't managed to get it fully functional.
Would it be possible to for you to build and run it fully in the docker environment and help iron out some of the issues?

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