Giter Club home page Giter Club logo

Comments (8)

marcbria avatar marcbria commented on September 8, 2024 1

In short:

  1. fork lucas repo.
  2. "git clone" your forked repo to your local computer.
  3. Make your chages.
  4. "git commit" to your local repo.
  5. "git push" to your forked repo.
  6. Visit github and the tool will ask you to create a PR with your changes.
  7. Delete your fork and local repo.

Here you have github's howto:
https://help.github.com/en/articles/creating-a-pull-request

from ojs.

lucasdiedrich avatar lucasdiedrich commented on September 8, 2024

Thats awesome @Potomac54, the error occurs because supervisor normally need an service to be running, and the exit of ojs-pre-start its not a running process and i never found and work around. Can you please make a PR with that?

Or i will integrate this as soon as i'm possible.

Thanks

from ojs.

Potomac54 avatar Potomac54 commented on September 8, 2024

Can you tell me how can I make pull request in your github ?

Do I need to be a member of your github ?
I tried to clone your github (php7-test branch), and then make a pull with my modifications, it doesn't accept my credentials

$ git push
Username for 'https://github.com': potomac
Password for 'https://[email protected]': 
remote: Permission to lucasdiedrich/ojs.git denied to Potomac.
fatal: unable to access 'https://github.com/lucasdiedrich/ojs.git/': The requested URL returned error: 403

then I tried to use the "pull request" feature of your github, but not sure to understand how it works

from ojs.

Potomac avatar Potomac commented on September 8, 2024

@marcbria @lucasdiedrich

Ok I forked lucas repo, I made the change in the "php7-test" branch, and I made a pull request :
#31

from ojs.

Potomac avatar Potomac commented on September 8, 2024

I notice a bug in my pull request, ojs-pre-start is not started when startsecs=0,

so the solution is to set this option to 1, like this :

[program:checkstart]
command=/usr/local/bin/ojs-pre-start
startsecs=1
exitcodes=0

from ojs.

lucasdiedrich avatar lucasdiedrich commented on September 8, 2024

@Potomac thanks for your help, unfortunetly your second PR included breaking code, i mentioned that over the PR, thanks.

from ojs.

Potomac avatar Potomac commented on September 8, 2024

@lucasdiedrich @marcbria : I fixed the wrong path in a new commit,

for the supervisor problem my fix doesn't really work, sometimes I get the message "fatal state", I think the main problem is that supervisord is designed to run daemons, not really simple bash scripts,

one solution would be to start ojs-pre-start with an init system like systemd, alpine linux uses a different init software, openRC : https://wiki.alpinelinux.org/wiki/Alpine_Linux_Init_System

another solution may be to use a cron script, with settings to run once ojs-pre-start script

from ojs.

lucasdiedrich avatar lucasdiedrich commented on September 8, 2024

@Potomac thats exactly the problem, supervisor is designed to run daemons, using alpine init system is the best option, as soon as possible going to look into this.

from ojs.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.