jicofo icon indicating copy to clipboard operation
jicofo copied to clipboard

Adding instructions for building the package from source

Open dmikushin opened this issue 4 years ago • 6 comments

It took me some time to get through https://github.com/jitsi/jicofo/issues/30 and figure out what steps are needed for successful command-line compilation. This PR proposes to add at least minimal directions into the README to simplify the developer experience, especially for those who are not native Java-speakers.

dmikushin avatar Sep 19 '19 21:09 dmikushin

Hi, thanks for your contribution! If you haven't already done so, could you please make sure you sign our CLA (https://jitsi.org/icla for individuals and https://jitsi.org/ccla for corporations)? We would unfortunately be unable to merge your patch unless we have that piece :(.

jitsi-jenkins avatar Sep 19 '19 21:09 jitsi-jenkins

For having just the jicofo.jar, why not just use mvn install the default command for every maven project ... ?

damencho avatar Sep 20 '19 06:09 damencho

Ant is just used for creating debian packages, the rest is in maven.

damencho avatar Sep 20 '19 06:09 damencho

I would also suggest using mvn install -Dassembly.skipAssembly=false to generate the zipped packages (which can be used manually as a replacement for /usr/share/jicofo in a debian installation).

bgrozev avatar Sep 25 '19 11:09 bgrozev

Thank you for your feedback! I've used ant specifically in order to make a local build for development purposes. That is, jicofo.jar is generated in the build folder and can be manually copied into existing Jitsi installation, without any packaging. Do you mean that mvn install can also be used for this?

ср, 25 сент. 2019 г. в 13:16, bgrozev [email protected]:

I would also suggest using mvn install -Dassembly.skipAssembly=false to generate the zipped packages (which can be used manually as a replacement for /usr/share/jicofo in a debian installation).

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/jitsi/jicofo/pull/387?email_source=notifications&email_token=ABLI7KUMSLF6OKSNJ3XIFULQLNCALA5CNFSM4IYQEVKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7RQ4LI#issuecomment-534973997, or mute the thread https://github.com/notifications/unsubscribe-auth/ABLI7KTPBQJJ2EKWHMJGNYDQLNCALANCNFSM4IYQEVKA .

dmikushin avatar Sep 25 '19 11:09 dmikushin

Yes, we recommend always using maven instead of ant. Just running mvn install should be enough to produce a new jicofo.jar. However, replacing jicofo.jar alone may not be sufficient because some of the dependencies may have changed (especially if you are working on some of the underlying libraries instead).

My procedure for manually building a custom version and deploying it to an existing remote installation is the following:

local$ mvn install -Dassembly.skipAssembly=false
local$ scp target/jicofo-linux-x64-1.1-SNAPSHOT.zip $remote:

remote$ unzip jicofo-linux-x64-1.1-SNAPSHOT.zip
remote$ sudo mv /usr/share/jicofo /usr/share/jicofo-orig
remote$ sudo ln -s $HOME/jicofo-linux-x64-1.1-SNAPSHOT /usr/share/jicofo
remote$ sudo service jicofo restart

bgrozev avatar Sep 25 '19 12:09 bgrozev

Thank you for your contribution and apologies for the lack of response. I cleaned up the readme and I think I picked up your suggestions.

bgrozev avatar Jan 06 '23 21:01 bgrozev