[jitsi-dev] [jitsi] Java Compiled Problem (#137)


#1

GoogleAPIClientToken cannot be resolved to a variable OAuth2TokenStore.java /jitsi/src/net/java/sip/communicator/impl/googlecontacts line 99

···

---
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi/issues/137


#2

If you run 'ant clean make' once it should create the file for you.

···

On Jul 27, 2015 5:39 AM, "Lycoris" <notifications@github.com> wrote:

GoogleAPIClientToken cannot be resolved to a variable
OAuth2TokenStore.java
/jitsi/src/net/java/sip/communicator/impl/googlecontacts line 99


Reply to this email directly or view it on GitHub
<https://github.com/jitsi/jitsi/issues/137>.

---
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi/issues/137#issuecomment-125098803


#3

It would be easier if that file would exist with the default null values. If not, at least put the generated file to .gitignore so the working directory isn't shown as dirty.

···

---
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi/issues/137#issuecomment-125103462


#4

OK.

This file is auto-updated by build.xml in order to provide id and secret needed to use oauth2 for accessing google api.

···

---
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi/issues/137#issuecomment-125112054


#5

@damencho would it work with build server configuration to make target "-pre-googlecontacts" a dependency for "clean" (instead of "compile")?

I think that would make more sense, make it more obvious for the developers. This is more a preparation of the source code (for compilation), rather than it is a preparation of the build process itself, isn't it. And we can add the generated file to .gitignore list. Would that work for you?

···

---
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi/issues/137#issuecomment-125344213


#6

Huh? Can't you just do it the same way as impl.version.NightlyBuildID?

···

---
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi/issues/137#issuecomment-125346295


#7

Hum, @lyubomir just give an idea, we can dump that file generation and read those values from default.properties and just make sure when those props are set we add those to the result file so we can package it in all OS. We just need to look in which ant target to update the.

···

---
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi/issues/137#issuecomment-125346346


#8

I thought it is currently as NightlyBuildID?

···

---
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi/issues/137#issuecomment-125346480


#9

Almost: NightlyBuildID.java is always there and under source control with a default value. It's overwritten at "make"-time.

···

---
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi/issues/137#issuecomment-125348928


#10

@cobratbq is it ok now?

···

---
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi/issues/137#issuecomment-125352080


#11

Closed #137.

···

---
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi/issues/137#event-376954734


#12

This was fixed in fba5aeba1ebeaea1942d949fe588d053b044e28a.

···

---
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi/issues/137#issuecomment-128933389