where to include JARs

[Vladimir Svoboda]

Hello,

I've got a question about importing JARs to Chromium, I've made changes at BaseChromiumApplication.java, some "import my.library...", question is to which gyp file/target should I add new dependencies? I've created targets at android_tools.gyp but don't know where to add dependencies to that targets. I've tried to add it to as dependency to target chrome_staging_java at chrome_apk.gyp but no luck.

Also I have a question if I need to add <classpathentry /> to .classpath file?

Thank you.

[d...@chromium.org]

chrome_java in chrome.gyp has a bunch of dependencies from android_tools. Did you try adding your dependency there?

--
--
Chromium Developers mailing list: chromi...@chromium.org
View archives, change email options, or unsubscribe:
http://groups.google.com/a/chromium.org/group/chromium-dev

[Vladimir Svoboda]

I tried it now, but still getting same error at "import my.lib..;"...

Dne pátek 11. září 2015 11:39:31 UTC+2 d...@chromium.org napsal(a):

[Vladimir Svoboda]

Edit: I'm building with "ninja -C out/Release chrome_public_apk"

I also tried to put it to chrome_android to target chrome_android_core with no luck...

Dne pátek 11. září 2015 11:45:37 UTC+2 Vladimir Svoboda napsal(a):

[d...@chromium.org]

Did you remember to run gyp again every time you change the gyp files?

[Vladimir Svoboda]

By that "run gyp again" you mean "gclient runhooks"? I've tried only "gclient runhooks" as Dominic sugested but still no luck...

Dne pátek 11. září 2015 12:38:14 UTC+2 d...@chromium.org napsal(a):

[d...@chromium.org]

Yes, that runs gyp. Sorry I'm not too familiar with adding entirely new libraries. Hopefully someone else can help better.

Otherwise I suppose your best bet is to pick a library there (android/support/v7/appcompat for example?), use code search and try to figure out from it what needs to be done.

And try to have a look in the out/ directory to make sure that all the intermediary targets are built. out/Debug/lib.java should have your jar at some point.

[Tommy Nyquist]

BaseChromiumApplication is part of //base so you would need to add the dependency to base_java.

[Thiago Farina]

On Fri, Sep 11, 2015 at 7:52 AM, Vladimir Svoboda <ladov...@gmail.com> wrote:

By that "run gyp again" you mean "gclient runhooks"? I've tried only "gclient runhooks" as Dominic sugested but still no luck...

runhooks runs a bunch of other things. If you just want to generate the build files again you can simply run build/gyp_chromium.

--

Thiago Farina

[Anton Vayvod]

Could you build with ninja -v and paste the full build error, including what intermediate target is being built.

--

--
Chromium Developers mailing list: chromi...@chromium.org
View archives, change email options, or unsubscribe:
http://groups.google.com/a/chromium.org/group/chromium-dev

To unsubscribe from this group and stop receiving emails from it, send an email to chromium-dev...@chromium.org.

[Anton Vayvod]

You may also run into Proguard issues, esp. if building the Release version - you might need to modify https://code.google.com/p/chromium/codesearch#chromium/src/chrome/android/java/proguard.flags to keep the classes from your library package.

[Vladimir Svoboda]

Hello again, I was out of office so I'm sorry for late response.

When I add dependencies to base_java as you suggest, running "gclient runhooks" or "build/gyp_chromium" gives no error. However when i run "ninja -v -C out/Release chrome_public_apk"

I got this error: 

lada@seamus:~/diplomka/chromium/src$ ninja -v -C out/Release chrome_public_apk
ninja: Entering directory `out/Release'
[1/638] cd ../../base; python ../build/android/gyp/javac.py "--classpath=/home/lada/diplomka/chromium/src/third_party/android_tools/sdk//platforms/android-22/android.jar \"../out/Release/lib.java/jsr_305_javalib.jar\"" "--src-gendirs=\"../out/Release/gen/enums/base_java_application_state/\" \"../out/Release/gen/enums/base_java_library_load_from_apk_status_codes/\" \"../out/Release/gen/enums/base_java_library_process_type/\" \"../out/Release/gen/enums/base_java_memory_pressure_level/\" \"../out/Release/gen/templates/base_native_libraries_gen/org/chromium/base/library_loader/\"" "--javac-includes=" "--chromium-code=1" "--jar-path=../out/Release/gen/base_java/base_java.jar" "--jar-excluded-classes=\"*/NativeLibraries.class\" \"*/R.class\" \"*/R##*.class\"" "--stamp=../out/Release/gen/base_java/compile.stamp" ../base/android/java/src/org/chromium/base/EventLog.java ../base/android/java/src/org/chromium/base/NativeCall.java ../base/android/java/src/org/chromium/base/PerfTraceEvent.java ../base/android/java/src/org/chromium/base/SystemMessageHandler.java ../base/android/java/src/org/chromium/base/PathService.java ../base/android/java/src/org/chromium/base/JNIUtils.java ../base/android/java/src/org/chromium/base/ImportantFileWriterAndroid.java ../base/android/java/src/org/chromium/base/ObserverList.java ../base/android/java/src/org/chromium/base/metrics/RecordUserAction.java ../base/android/java/src/org/chromium/base/metrics/RecordHistogram.java ../base/android/java/src/org/chromium/base/CalledByNative.java ../base/android/java/src/org/chromium/base/VisibleForTesting.java ../base/android/java/src/org/chromium/base/ApkAssets.java ../base/android/java/src/org/chromium/base/TraceEvent.java ../base/android/java/src/org/chromium/base/ActivityState.java ../base/android/java/src/org/chromium/base/ContentUriUtils.java ../base/android/java/src/org/chromium/base/SecureRandomInitializer.java ../base/android/java/src/org/chromium/base/BaseChromiumApplication.java ../base/android/java/src/org/chromium/base/Log.java ../base/android/java/src/org/chromium/base/JNINamespace.java ../base/android/java/src/org/chromium/base/BaseSwitches.java ../base/android/java/src/org/chromium/base/library_loader/LoaderErrors.java ../base/android/java/src/org/chromium/base/library_loader/LibraryLoaderHelper.java ../base/android/java/src/org/chromium/base/library_loader/ProcessInitException.java ../base/android/java/src/org/chromium/base/library_loader/LibraryLoader.java ../base/android/java/src/org/chromium/base/library_loader/Linker.java ../base/android/java/src/org/chromium/base/ResourceExtractor.java ../base/android/java/src/org/chromium/base/SysUtils.java ../base/android/java/src/org/chromium/base/AnimationFrameTimeHistogram.java ../base/android/java/src/org/chromium/base/JavaHandlerThread.java ../base/android/java/src/org/chromium/base/annotations/AccessedByNative.java ../base/android/java/src/org/chromium/base/annotations/NoSideEffects.java ../base/android/java/src/org/chromium/base/annotations/SuppressFBWarnings.java ../base/android/java/src/org/chromium/base/annotations/UsedByReflection.java ../base/android/java/src/org/chromium/base/annotations/RemovableInRelease.java ../base/android/java/src/org/chromium/base/annotations/CalledByNativeUnchecked.java ../base/android/java/src/org/chromium/base/MemoryPressureListener.java ../base/android/java/src/org/chromium/base/CpuFeatures.java ../base/android/java/src/org/chromium/base/PackageUtils.java ../base/android/java/src/org/chromium/base/PowerStatusReceiver.java ../base/android/java/src/org/chromium/base/CollectionUtil.java ../base/android/java/src/org/chromium/base/ThreadUtils.java ../base/android/java/src/org/chromium/base/ApplicationStatus.java ../base/android/java/src/org/chromium/base/LocaleUtils.java ../base/android/java/src/org/chromium/base/CommandLine.java ../base/android/java/src/org/chromium/base/JNIAdditionalImport.java ../base/android/java/src/org/chromium/base/FieldTrialList.java ../base/android/java/src/org/chromium/base/ApiCompatibilityUtils.java ../base/android/java/src/org/chromium/base/BuildInfo.java ../base/android/java/src/org/chromium/base/NativeClassQualifiedName.java ../base/android/java/src/org/chromium/base/PathUtils.java ../base/android/java/src/org/chromium/base/PowerMonitor.java
FAILED: cd ../../base; python ../build/android/gyp/javac.py "--classpath=/home/lada/diplomka/chromium/src/third_party/android_tools/sdk//platforms/android-22/android.jar \"../out/Release/lib.java/jsr_305_javalib.jar\"" "--src-gendirs=\"../out/Release/gen/enums/base_java_application_state/\" \"../out/Release/gen/enums/base_java_library_load_from_apk_status_codes/\" \"../out/Release/gen/enums/base_java_library_process_type/\" \"../out/Release/gen/enums/base_java_memory_pressure_level/\" \"../out/Release/gen/templates/base_native_libraries_gen/org/chromium/base/library_loader/\"" "--javac-includes=" "--chromium-code=1" "--jar-path=../out/Release/gen/base_java/base_java.jar" "--jar-excluded-classes=\"*/NativeLibraries.class\" \"*/R.class\" \"*/R##*.class\"" "--stamp=../out/Release/gen/base_java/compile.stamp" ../base/android/java/src/org/chromium/base/EventLog.java ../base/android/java/src/org/chromium/base/NativeCall.java ../base/android/java/src/org/chromium/base/PerfTraceEvent.java ../base/android/java/src/org/chromium/base/SystemMessageHandler.java ../base/android/java/src/org/chromium/base/PathService.java ../base/android/java/src/org/chromium/base/JNIUtils.java ../base/android/java/src/org/chromium/base/ImportantFileWriterAndroid.java ../base/android/java/src/org/chromium/base/ObserverList.java ../base/android/java/src/org/chromium/base/metrics/RecordUserAction.java ../base/android/java/src/org/chromium/base/metrics/RecordHistogram.java ../base/android/java/src/org/chromium/base/CalledByNative.java ../base/android/java/src/org/chromium/base/VisibleForTesting.java ../base/android/java/src/org/chromium/base/ApkAssets.java ../base/android/java/src/org/chromium/base/TraceEvent.java ../base/android/java/src/org/chromium/base/ActivityState.java ../base/android/java/src/org/chromium/base/ContentUriUtils.java ../base/android/java/src/org/chromium/base/SecureRandomInitializer.java ../base/android/java/src/org/chromium/base/BaseChromiumApplication.java ../base/android/java/src/org/chromium/base/Log.java ../base/android/java/src/org/chromium/base/JNINamespace.java ../base/android/java/src/org/chromium/base/BaseSwitches.java ../base/android/java/src/org/chromium/base/library_loader/LoaderErrors.java ../base/android/java/src/org/chromium/base/library_loader/LibraryLoaderHelper.java ../base/android/java/src/org/chromium/base/library_loader/ProcessInitException.java ../base/android/java/src/org/chromium/base/library_loader/LibraryLoader.java ../base/android/java/src/org/chromium/base/library_loader/Linker.java ../base/android/java/src/org/chromium/base/ResourceExtractor.java ../base/android/java/src/org/chromium/base/SysUtils.java ../base/android/java/src/org/chromium/base/AnimationFrameTimeHistogram.java ../base/android/java/src/org/chromium/base/JavaHandlerThread.java ../base/android/java/src/org/chromium/base/annotations/AccessedByNative.java ../base/android/java/src/org/chromium/base/annotations/NoSideEffects.java ../base/android/java/src/org/chromium/base/annotations/SuppressFBWarnings.java ../base/android/java/src/org/chromium/base/annotations/UsedByReflection.java ../base/android/java/src/org/chromium/base/annotations/RemovableInRelease.java ../base/android/java/src/org/chromium/base/annotations/CalledByNativeUnchecked.java ../base/android/java/src/org/chromium/base/MemoryPressureListener.java ../base/android/java/src/org/chromium/base/CpuFeatures.java ../base/android/java/src/org/chromium/base/PackageUtils.java ../base/android/java/src/org/chromium/base/PowerStatusReceiver.java ../base/android/java/src/org/chromium/base/CollectionUtil.java ../base/android/java/src/org/chromium/base/ThreadUtils.java ../base/android/java/src/org/chromium/base/ApplicationStatus.java ../base/android/java/src/org/chromium/base/LocaleUtils.java ../base/android/java/src/org/chromium/base/CommandLine.java ../base/android/java/src/org/chromium/base/JNIAdditionalImport.java ../base/android/java/src/org/chromium/base/FieldTrialList.java ../base/android/java/src/org/chromium/base/ApiCompatibilityUtils.java ../base/android/java/src/org/chromium/base/BuildInfo.java ../base/android/java/src/org/chromium/base/NativeClassQualifiedName.java ../base/android/java/src/org/chromium/base/PathUtils.java ../base/android/java/src/org/chromium/base/PowerMonitor.java
Traceback (most recent call last):
  File "../build/android/gyp/javac.py", line 280, in <module>
    sys.exit(main(sys.argv[1:]))
  File "../build/android/gyp/javac.py", line 244, in main
    java_files)
  File "../build/android/gyp/javac.py", line 104, in DoJavac
    input_strings=javac_cmd)
  File "/home/lada/diplomka/chromium/src/build/android/gyp/util/md5_check.py", line 30, in CallAndRecordIfStale
    function()
  File "../build/android/gyp/javac.py", line 97, in Compile
    stderr_filter=ColorJavacOutput)
  File "/home/lada/diplomka/chromium/src/build/android/gyp/util/build_utils.py", line 158, in CheckOutput
    raise CalledProcessError(cwd, args, stdout + stderr)
util.build_utils.CalledProcessError: Command failed: ( cd /home/lada/diplomka/chromium/src/base; javac -g -encoding UTF-8 -source 1.7 -target 1.7 -classpath /home/lada/diplomka/chromium/src/third_party/android_tools/sdk//platforms/android-22/android.jar:../out/Release/lib.java/jsr_305_javalib.jar -d /tmp/tmpiHfZjE/classes -Xlint:unchecked ../base/android/java/src/org/chromium/base/EventLog.java ../base/android/java/src/org/chromium/base/NativeCall.java ../base/android/java/src/org/chromium/base/PerfTraceEvent.java ../base/android/java/src/org/chromium/base/SystemMessageHandler.java ../base/android/java/src/org/chromium/base/PathService.java ../base/android/java/src/org/chromium/base/JNIUtils.java ../base/android/java/src/org/chromium/base/ImportantFileWriterAndroid.java ../base/android/java/src/org/chromium/base/ObserverList.java ../base/android/java/src/org/chromium/base/metrics/RecordUserAction.java ../base/android/java/src/org/chromium/base/metrics/RecordHistogram.java ../base/android/java/src/org/chromium/base/CalledByNative.java ../base/android/java/src/org/chromium/base/VisibleForTesting.java ../base/android/java/src/org/chromium/base/ApkAssets.java ../base/android/java/src/org/chromium/base/TraceEvent.java ../base/android/java/src/org/chromium/base/ActivityState.java ../base/android/java/src/org/chromium/base/ContentUriUtils.java ../base/android/java/src/org/chromium/base/SecureRandomInitializer.java ../base/android/java/src/org/chromium/base/BaseChromiumApplication.java ../base/android/java/src/org/chromium/base/Log.java ../base/android/java/src/org/chromium/base/JNINamespace.java ../base/android/java/src/org/chromium/base/BaseSwitches.java ../base/android/java/src/org/chromium/base/library_loader/LoaderErrors.java ../base/android/java/src/org/chromium/base/library_loader/LibraryLoaderHelper.java ../base/android/java/src/org/chromium/base/library_loader/ProcessInitException.java ../base/android/java/src/org/chromium/base/library_loader/LibraryLoader.java ../base/android/java/src/org/chromium/base/library_loader/Linker.java ../base/android/java/src/org/chromium/base/ResourceExtractor.java ../base/android/java/src/org/chromium/base/SysUtils.java ../base/android/java/src/org/chromium/base/AnimationFrameTimeHistogram.java ../base/android/java/src/org/chromium/base/JavaHandlerThread.java ../base/android/java/src/org/chromium/base/annotations/AccessedByNative.java ../base/android/java/src/org/chromium/base/annotations/NoSideEffects.java ../base/android/java/src/org/chromium/base/annotations/SuppressFBWarnings.java ../base/android/java/src/org/chromium/base/annotations/UsedByReflection.java ../base/android/java/src/org/chromium/base/annotations/RemovableInRelease.java ../base/android/java/src/org/chromium/base/annotations/CalledByNativeUnchecked.java ../base/android/java/src/org/chromium/base/MemoryPressureListener.java ../base/android/java/src/org/chromium/base/CpuFeatures.java ../base/android/java/src/org/chromium/base/PackageUtils.java ../base/android/java/src/org/chromium/base/PowerStatusReceiver.java ../base/android/java/src/org/chromium/base/CollectionUtil.java ../base/android/java/src/org/chromium/base/ThreadUtils.java ../base/android/java/src/org/chromium/base/ApplicationStatus.java ../base/android/java/src/org/chromium/base/LocaleUtils.java ../base/android/java/src/org/chromium/base/CommandLine.java ../base/android/java/src/org/chromium/base/JNIAdditionalImport.java ../base/android/java/src/org/chromium/base/FieldTrialList.java ../base/android/java/src/org/chromium/base/ApiCompatibilityUtils.java ../base/android/java/src/org/chromium/base/BuildInfo.java ../base/android/java/src/org/chromium/base/NativeClassQualifiedName.java ../base/android/java/src/org/chromium/base/PathUtils.java ../base/android/java/src/org/chromium/base/PowerMonitor.java ../out/Release/gen/enums/base_java_application_state/org/chromium/base/ApplicationState.java ../out/Release/gen/enums/base_java_library_load_from_apk_status_codes/org/chromium/base/library_loader/LibraryLoadFromApkStatusCodes.java ../out/Release/gen/enums/base_java_library_process_type/org/chromium/base/library_loader/LibraryProcessType.java ../out/Release/gen/enums/base_java_memory_pressure_level/org/chromium/base/MemoryPressureLevel.java ../out/Release/gen/templates/base_native_libraries_gen/org/chromium/base/library_loader/NativeLibraries.java )
../base/android/java/src/org/chromium/base/BaseChromiumApplication.java:20: error: package at.cyan.smartprotection.internet does not exist
import at.cyan.smartprotection.internet.UrlCheckResult;
                                       ^
../base/android/java/src/org/chromium/base/BaseChromiumApplication.java:21: error: package at.cyan.smartprotection.shared does not exist
import at.cyan.smartprotection.shared.ControlUnit;
                                     ^
../base/android/java/src/org/chromium/base/BaseChromiumApplication.java:39: error: cannot find symbol
    private final ControlUnit controlUnit;
                  ^
  symbol:   class ControlUnit
  location: class BaseChromiumApplication
3 errors

ninja: build stopped: subcommand failed.



Note that that missing package is included in JARs /src/base/base.gyp like this:
{
          # GN: //base:base_java
          'target_name': 'base_java',
          'type': 'none',
          'variables': {
            'java_in_dir': '../base/android/java',
            'jar_excluded_classes': [ '*/NativeLibraries.class' ],
          },
          'dependencies': [
            'base_java_application_state',
            'base_java_library_load_from_apk_status_codes',
            'base_java_library_process_type',
            'base_java_memory_pressure_level',
            'base_native_libraries_gen',
            '../third_party/jsr-305/jsr-305.gyp:jsr_305_javalib',
                                    #cyan
        '../third_party/android_tools/android_tools.gyp:gson',
        '../third_party/android_tools/android_tools.gyp:cyanAntiVirus4Android',
        '../third_party/android_tools/android_tools.gyp:cyanproviderintegrationapi',
        '../third_party/android_tools/android_tools.gyp:cyanproviderintegrationimplcyan-release',
        '../third_party/android_tools/android_tools.gyp:cyanSmartProtection-release',
        '../third_party/android_tools/android_tools.gyp:play-services-location',
          ],
          'includes': [ '../build/java.gypi' ],
 }

and targets are defined at android_tools.gyp  like this:

{
      'target_name': 'play-services-location',
      'type' : 'none',
      'variables': {
        'jar_path': '../third_party/cyan/play-services-location/play-services-location-7.5.0.jar',
      },
    },
    {
      'target_name': 'gson',
      'type' : 'none',
      'variables': {
        'jar_path': '../third_party/cyan/gson/gson-2.3.1.jar',
      },
    },
    {
      'target_name': 'cyanAntiVirus4Android',
      'type' : 'none',
      'variables': {
        'jar_path': '../third_party/cyan/cyanAntiVirus4Android-release/cyanAntiVirus4Android-release.jar',
      },
    },
    {
      'target_name': 'cyanproviderintegrationapi',
      'type' : 'none',
      'variables': {
        'jar_path': '../third_party/cyan/cyanproviderintegrationapi/cyanproviderintegrationapi.jar',
      },
    },
    {
      'target_name': 'cyanproviderintegrationimplcyan-release',
      'type' : 'none',
      'variables': {
        'jar_path': '../third_party/cyan/cyanproviderintegrationimplcyan-release/cyanproviderintegrationimplcyan-release.jar',
      },
    },
    {
      'target_name': 'cyanSmartProtection-release',
      'type' : 'none',
      'variables': {
        'jar_path': '../third_party/cyan/cyanSmartProtection-release/cyanSmartProtection-release.jar',
      },
    }


Anything I missed?


Dne pátek 11. září 2015 17:06:04 UTC+2 Tommy Nyquist napsal(a):

[d...@chromium.org]

You're missing  the "includes" that adds the target description:

    {

      'target_name': 'foo',

      'type' : 'none',

      'variables': {

        'jar_path': 'foo/foo.jar',

      },

      'includes': ['../../build/java_prebuilt.gypi'],

    },

[d...@chromium.org]

My understanding of GYP is that targets do nothing by themselves. the includes parameter describes how your arguments should be used to build the various target you are looking for, So includes declares what type your target is. Right now, you just set variables but didn't do anything with them. So that's most likely one of the reasons why the jars were not picked up. Hopefully that's also the only reason.

[Anton Vayvod]

I'd rather say that 'target's do describe how to build something but most 'target's use 'includes' and set input variables to specify common build steps to avoid redundancy :) If you expand the 'includes' with the .gypi you will get the complete 'target' that's being processed by gyp.

To unsubscribe from this group and stop receiving emails from it, send an email to chromium-dev...@chromium.org.

[Vladimir Svoboda]

I was wrong, I had comented out that imports so I though it's working, but it's nearly the same :{ my files look like this:

base.gyp:

{
          # GN: //base:base_java
          'target_name': 'base_java',
          'type': 'none',
          'variables': {
            'java_in_dir': '../base/android/java',
            'jar_excluded_classes': [ '*/NativeLibraries.class' ],
          },
          'dependencies': [
            'base_java_application_state',
            'base_java_library_load_from_apk_status_codes',
            'base_java_library_process_type',
            'base_java_memory_pressure_level',
            'base_native_libraries_gen',
            '../third_party/jsr-305/jsr-305.gyp:jsr_305_javalib',
                                    #cyan
        '../third_party/android_tools/android_tools.gyp:gson',
        '../third_party/android_tools/android_tools.gyp:cyanAntiVirus4Android',
        '../third_party/android_tools/android_tools.gyp:cyanproviderintegrationapi',
        '../third_party/android_tools/android_tools.gyp:cyanproviderintegrationimplcyan-release',
        '../third_party/android_tools/android_tools.gyp:cyanSmartProtection-release',
        '../third_party/android_tools/android_tools.gyp:play-services-location',
          ],
          'includes': [ '../build/java.gypi' ],

        },


my targets at android_tools.gyp:

{
      'target_name': 'play-services-location',
      'type' : 'none',
      'variables': {

        'jar_path': '../cyan/play-services-location/play-services-location-7.5.0.jar',

      },
     'includes': ['../../build/java_prebuilt.gypi'],
    },

    {
      'target_name': 'gson',
      'type' : 'none',
      'variables': {

        'jar_path': '../cyan/gson/gson-2.3.1.jar',

      },
     'includes': ['../../build/java_prebuilt.gypi'],
    },

    {
      'target_name': 'cyanAntiVirus4Android',
      'type' : 'none',
      'variables': {

        'jar_path': '../cyan/cyanAntiVirus4Android-release/cyanAntiVirus4Android-release.jar',

      },
     'includes': ['../../build/java_prebuilt.gypi'],
    },

    {
      'target_name': 'cyanproviderintegrationapi',
      'type' : 'none',
      'variables': {

        'jar_path': '../cyan/cyanproviderintegrationapi/cyanproviderintegrationapi.jar',

      },
     'includes': ['../../build/java_prebuilt.gypi'],
    },

    {
      'target_name': 'cyanproviderintegrationimplcyan-release',
      'type' : 'none',
      'variables': {

        'jar_path': '../cyan/cyanproviderintegrationimplcyan-release/cyanproviderintegrationimplcyan-release.jar',

      },
     'includes': ['../../build/java_prebuilt.gypi'],
    },

    {
      'target_name': 'cyanSmartProtection-release',
      'type' : 'none',
      'variables': {

        'jar_path': '../cyan/cyanSmartProtection-release/cyanSmartProtection-release.jar',
      },
     'includes': ['../../build/java_prebuilt.gypi'],
    }

and I get this error when using ninja (it has changed a little, now it atleast contains that JARs at -classpath):

lada@seamus:~/diplomka/chromium/src$ ninja -v -C out/Release chrome_public_apk
ninja: Entering directory `out/Release'

[1/638] cd ../../base; python ../build/android/gyp/javac.py "--classpath=/home/lada/diplomka/chromium/src/third_party/android_tools/sdk//platforms/android-22/android.jar \"../out/Release/lib.java/jsr_305_javalib.jar\" ../third_party/cyan/gson/gson-2.3.1.jar ../third_party/cyan/cyanAntiVirus4Android-release/cyanAntiVirus4Android-release.jar ../third_party/cyan/cyanproviderintegrationapi/cyanproviderintegrationapi.jar ../third_party/cyan/cyanproviderintegrationimplcyan-release/cyanproviderintegrationimplcyan-release.jar ../third_party/cyan/cyanSmartProtection-release/cyanSmartProtection-release.jar ../third_party/cyan/play-services-location/play-services-location-7.5.0.jar" "--src-gendirs=\"../out/Release/gen/enums/base_java_application_state/\" \"../out/Release/gen/enums/base_java_library_load_from_apk_status_codes/\" \"../out/Release/gen/enums/base_java_library_process_type/\" \"../out/Release/gen/enums/base_java_memory_pressure_level/\" \"../out/Release/gen/templates/base_native_libraries_gen/org/chromium/base/library_loader/\"" "--javac-includes=" "--chromium-code=1" "--jar-path=../out/Release/gen/base_java/base_java.jar" "--jar-excluded-classes=\"*/NativeLibraries.class\" \"*/R.class\" \"*/R##*.class\"" "--stamp=../out/Release/gen/base_java/compile.stamp" ../base/android/java/src/org/chromium/base/EventLog.java ../base/android/java/src/org/chromium/base/NativeCall.java ../base/android/java/src/org/chromium/base/PerfTraceEvent.java ../base/android/java/src/org/chromium/base/SystemMessageHandler.java ../base/android/java/src/org/chromium/base/PathService.java ../base/android/java/src/org/chromium/base/JNIUtils.java ../base/android/java/src/org/chromium/base/ImportantFileWriterAndroid.java ../base/android/java/src/org/chromium/base/ObserverList.java ../base/android/java/src/org/chromium/base/metrics/RecordUserAction.java ../base/android/java/src/org/chromium/base/metrics/RecordHistogram.java ../base/android/java/src/org/chromium/base/CalledByNative.java ../base/android/java/src/org/chromium/base/VisibleForTesting.java ../base/android/java/src/org/chromium/base/ApkAssets.java ../base/android/java/src/org/chromium/base/TraceEvent.java ../base/android/java/src/org/chromium/base/ActivityState.java ../base/android/java/src/org/chromium/base/ContentUriUtils.java ../base/android/java/src/org/chromium/base/SecureRandomInitializer.java ../base/android/java/src/org/chromium/base/BaseChromiumApplication.java ../base/android/java/src/org/chromium/base/Log.java ../base/android/java/src/org/chromium/base/JNINamespace.java ../base/android/java/src/org/chromium/base/BaseSwitches.java ../base/android/java/src/org/chromium/base/library_loader/LoaderErrors.java ../base/android/java/src/org/chromium/base/library_loader/LibraryLoaderHelper.java ../base/android/java/src/org/chromium/base/library_loader/ProcessInitException.java ../base/android/java/src/org/chromium/base/library_loader/LibraryLoader.java ../base/android/java/src/org/chromium/base/library_loader/Linker.java ../base/android/java/src/org/chromium/base/ResourceExtractor.java ../base/android/java/src/org/chromium/base/SysUtils.java ../base/android/java/src/org/chromium/base/AnimationFrameTimeHistogram.java ../base/android/java/src/org/chromium/base/JavaHandlerThread.java ../base/android/java/src/org/chromium/base/annotations/AccessedByNative.java ../base/android/java/src/org/chromium/base/annotations/NoSideEffects.java ../base/android/java/src/org/chromium/base/annotations/SuppressFBWarnings.java ../base/android/java/src/org/chromium/base/annotations/UsedByReflection.java ../base/android/java/src/org/chromium/base/annotations/RemovableInRelease.java ../base/android/java/src/org/chromium/base/annotations/CalledByNativeUnchecked.java ../base/android/java/src/org/chromium/base/MemoryPressureListener.java ../base/android/java/src/org/chromium/base/CpuFeatures.java ../base/android/java/src/org/chromium/base/PackageUtils.java ../base/android/java/src/org/chromium/base/PowerStatusReceiver.java ../base/android/java/src/org/chromium/base/CollectionUtil.java ../base/android/java/src/org/chromium/base/ThreadUtils.java ../base/android/java/src/org/chromium/base/ApplicationStatus.java ../base/android/java/src/org/chromium/base/LocaleUtils.java ../base/android/java/src/org/chromium/base/CommandLine.java ../base/android/java/src/org/chromium/base/JNIAdditionalImport.java ../base/android/java/src/org/chromium/base/FieldTrialList.java ../base/android/java/src/org/chromium/base/ApiCompatibilityUtils.java ../base/android/java/src/org/chromium/base/BuildInfo.java ../base/android/java/src/org/chromium/base/NativeClassQualifiedName.java ../base/android/java/src/org/chromium/base/PathUtils.java ../base/android/java/src/org/chromium/base/PowerMonitor.java
FAILED: cd ../../base; python ../build/android/gyp/javac.py "--classpath=/home/lada/diplomka/chromium/src/third_party/android_tools/sdk//platforms/android-22/android.jar \"../out/Release/lib.java/jsr_305_javalib.jar\" ../third_party/cyan/gson/gson-2.3.1.jar ../third_party/cyan/cyanAntiVirus4Android-release/cyanAntiVirus4Android-release.jar ../third_party/cyan/cyanproviderintegrationapi/cyanproviderintegrationapi.jar ../third_party/cyan/cyanproviderintegrationimplcyan-release/cyanproviderintegrationimplcyan-release.jar ../third_party/cyan/cyanSmartProtection-release/cyanSmartProtection-release.jar ../third_party/cyan/play-services-location/play-services-location-7.5.0.jar" "--src-gendirs=\"../out/Release/gen/enums/base_java_application_state/\" \"../out/Release/gen/enums/base_java_library_load_from_apk_status_codes/\" \"../out/Release/gen/enums/base_java_library_process_type/\" \"../out/Release/gen/enums/base_java_memory_pressure_level/\" \"../out/Release/gen/templates/base_native_libraries_gen/org/chromium/base/library_loader/\"" "--javac-includes=" "--chromium-code=1" "--jar-path=../out/Release/gen/base_java/base_java.jar" "--jar-excluded-classes=\"*/NativeLibraries.class\" \"*/R.class\" \"*/R##*.class\"" "--stamp=../out/Release/gen/base_java/compile.stamp" ../base/android/java/src/org/chromium/base/EventLog.java ../base/android/java/src/org/chromium/base/NativeCall.java ../base/android/java/src/org/chromium/base/PerfTraceEvent.java ../base/android/java/src/org/chromium/base/SystemMessageHandler.java ../base/android/java/src/org/chromium/base/PathService.java ../base/android/java/src/org/chromium/base/JNIUtils.java ../base/android/java/src/org/chromium/base/ImportantFileWriterAndroid.java ../base/android/java/src/org/chromium/base/ObserverList.java ../base/android/java/src/org/chromium/base/metrics/RecordUserAction.java ../base/android/java/src/org/chromium/base/metrics/RecordHistogram.java ../base/android/java/src/org/chromium/base/CalledByNative.java ../base/android/java/src/org/chromium/base/VisibleForTesting.java ../base/android/java/src/org/chromium/base/ApkAssets.java ../base/android/java/src/org/chromium/base/TraceEvent.java ../base/android/java/src/org/chromium/base/ActivityState.java ../base/android/java/src/org/chromium/base/ContentUriUtils.java ../base/android/java/src/org/chromium/base/SecureRandomInitializer.java ../base/android/java/src/org/chromium/base/BaseChromiumApplication.java ../base/android/java/src/org/chromium/base/Log.java ../base/android/java/src/org/chromium/base/JNINamespace.java ../base/android/java/src/org/chromium/base/BaseSwitches.java ../base/android/java/src/org/chromium/base/library_loader/LoaderErrors.java ../base/android/java/src/org/chromium/base/library_loader/LibraryLoaderHelper.java ../base/android/java/src/org/chromium/base/library_loader/ProcessInitException.java ../base/android/java/src/org/chromium/base/library_loader/LibraryLoader.java ../base/android/java/src/org/chromium/base/library_loader/Linker.java ../base/android/java/src/org/chromium/base/ResourceExtractor.java ../base/android/java/src/org/chromium/base/SysUtils.java ../base/android/java/src/org/chromium/base/AnimationFrameTimeHistogram.java ../base/android/java/src/org/chromium/base/JavaHandlerThread.java ../base/android/java/src/org/chromium/base/annotations/AccessedByNative.java ../base/android/java/src/org/chromium/base/annotations/NoSideEffects.java ../base/android/java/src/org/chromium/base/annotations/SuppressFBWarnings.java ../base/android/java/src/org/chromium/base/annotations/UsedByReflection.java ../base/android/java/src/org/chromium/base/annotations/RemovableInRelease.java ../base/android/java/src/org/chromium/base/annotations/CalledByNativeUnchecked.java ../base/android/java/src/org/chromium/base/MemoryPressureListener.java ../base/android/java/src/org/chromium/base/CpuFeatures.java ../base/android/java/src/org/chromium/base/PackageUtils.java ../base/android/java/src/org/chromium/base/PowerStatusReceiver.java ../base/android/java/src/org/chromium/base/CollectionUtil.java ../base/android/java/src/org/chromium/base/ThreadUtils.java ../base/android/java/src/org/chromium/base/ApplicationStatus.java ../base/android/java/src/org/chromium/base/LocaleUtils.java ../base/android/java/src/org/chromium/base/CommandLine.java ../base/android/java/src/org/chromium/base/JNIAdditionalImport.java ../base/android/java/src/org/chromium/base/FieldTrialList.java ../base/android/java/src/org/chromium/base/ApiCompatibilityUtils.java ../base/android/java/src/org/chromium/base/BuildInfo.java ../base/android/java/src/org/chromium/base/NativeClassQualifiedName.java ../base/android/java/src/org/chromium/base/PathUtils.java ../base/android/java/src/org/chromium/base/PowerMonitor.java

Traceback (most recent call last):
  File "../build/android/gyp/javac.py", line 280, in <module>
    sys.exit(main(sys.argv[1:]))
  File "../build/android/gyp/javac.py", line 244, in main
    java_files)
  File "../build/android/gyp/javac.py", line 104, in DoJavac
    input_strings=javac_cmd)
  File "/home/lada/diplomka/chromium/src/build/android/gyp/util/md5_check.py", line 30, in CallAndRecordIfStale
    function()
  File "../build/android/gyp/javac.py", line 97, in Compile
    stderr_filter=ColorJavacOutput)
  File "/home/lada/diplomka/chromium/src/build/android/gyp/util/build_utils.py", line 158, in CheckOutput
    raise CalledProcessError(cwd, args, stdout + stderr)

util.build_utils.CalledProcessError: Command failed: ( cd /home/lada/diplomka/chromium/src/base; javac -g -encoding UTF-8 -source 1.7 -target 1.7 -classpath /home/lada/diplomka/chromium/src/third_party/android_tools/sdk//platforms/android-22/android.jar:../out/Release/lib.java/jsr_305_javalib.jar:../third_party/cyan/gson/gson-2.3.1.jar:../third_party/cyan/cyanAntiVirus4Android-release/cyanAntiVirus4Android-release.jar:../third_party/cyan/cyanproviderintegrationapi/cyanproviderintegrationapi.jar:../third_party/cyan/cyanproviderintegrationimplcyan-release/cyanproviderintegrationimplcyan-release.jar:../third_party/cyan/cyanSmartProtection-release/cyanSmartProtection-release.jar:../third_party/cyan/play-services-location/play-services-location-7.5.0.jar -d /tmp/tmpqTiHjn/classes -Xlint:unchecked ../base/android/java/src/org/chromium/base/EventLog.java ../base/android/java/src/org/chromium/base/NativeCall.java ../base/android/java/src/org/chromium/base/PerfTraceEvent.java ../base/android/java/src/org/chromium/base/SystemMessageHandler.java ../base/android/java/src/org/chromium/base/PathService.java ../base/android/java/src/org/chromium/base/JNIUtils.java ../base/android/java/src/org/chromium/base/ImportantFileWriterAndroid.java ../base/android/java/src/org/chromium/base/ObserverList.java ../base/android/java/src/org/chromium/base/metrics/RecordUserAction.java ../base/android/java/src/org/chromium/base/metrics/RecordHistogram.java ../base/android/java/src/org/chromium/base/CalledByNative.java ../base/android/java/src/org/chromium/base/VisibleForTesting.java ../base/android/java/src/org/chromium/base/ApkAssets.java ../base/android/java/src/org/chromium/base/TraceEvent.java ../base/android/java/src/org/chromium/base/ActivityState.java ../base/android/java/src/org/chromium/base/ContentUriUtils.java ../base/android/java/src/org/chromium/base/SecureRandomInitializer.java ../base/android/java/src/org/chromium/base/BaseChromiumApplication.java ../base/android/java/src/org/chromium/base/Log.java ../base/android/java/src/org/chromium/base/JNINamespace.java ../base/android/java/src/org/chromium/base/BaseSwitches.java ../base/android/java/src/org/chromium/base/library_loader/LoaderErrors.java ../base/android/java/src/org/chromium/base/library_loader/LibraryLoaderHelper.java ../base/android/java/src/org/chromium/base/library_loader/ProcessInitException.java ../base/android/java/src/org/chromium/base/library_loader/LibraryLoader.java ../base/android/java/src/org/chromium/base/library_loader/Linker.java ../base/android/java/src/org/chromium/base/ResourceExtractor.java ../base/android/java/src/org/chromium/base/SysUtils.java ../base/android/java/src/org/chromium/base/AnimationFrameTimeHistogram.java ../base/android/java/src/org/chromium/base/JavaHandlerThread.java ../base/android/java/src/org/chromium/base/annotations/AccessedByNative.java ../base/android/java/src/org/chromium/base/annotations/NoSideEffects.java ../base/android/java/src/org/chromium/base/annotations/SuppressFBWarnings.java ../base/android/java/src/org/chromium/base/annotations/UsedByReflection.java ../base/android/java/src/org/chromium/base/annotations/RemovableInRelease.java ../base/android/java/src/org/chromium/base/annotations/CalledByNativeUnchecked.java ../base/android/java/src/org/chromium/base/MemoryPressureListener.java ../base/android/java/src/org/chromium/base/CpuFeatures.java ../base/android/java/src/org/chromium/base/PackageUtils.java ../base/android/java/src/org/chromium/base/PowerStatusReceiver.java ../base/android/java/src/org/chromium/base/CollectionUtil.java ../base/android/java/src/org/chromium/base/ThreadUtils.java ../base/android/java/src/org/chromium/base/ApplicationStatus.java ../base/android/java/src/org/chromium/base/LocaleUtils.java ../base/android/java/src/org/chromium/base/CommandLine.java ../base/android/java/src/org/chromium/base/JNIAdditionalImport.java ../base/android/java/src/org/chromium/base/FieldTrialList.java ../base/android/java/src/org/chromium/base/ApiCompatibilityUtils.java ../base/android/java/src/org/chromium/base/BuildInfo.java ../base/android/java/src/org/chromium/base/NativeClassQualifiedName.java ../base/android/java/src/org/chromium/base/PathUtils.java ../base/android/java/src/org/chromium/base/PowerMonitor.java ../out/Release/gen/enums/base_java_application_state/org/chromium/base/ApplicationState.java ../out/Release/gen/enums/base_java_library_load_from_apk_status_codes/org/chromium/base/library_loader/LibraryLoadFromApkStatusCodes.java ../out/Release/gen/enums/base_java_library_process_type/org/chromium/base/library_loader/LibraryProcessType.java ../out/Release/gen/enums/base_java_memory_pressure_level/org/chromium/base/MemoryPressureLevel.java ../out/Release/gen/templates/base_native_libraries_gen/org/chromium/base/library_loader/NativeLibraries.java )
../base/android/java/src/org/chromium/base/BaseChromiumApplication.java:23: error: package at.cyan.smartprotection.internet does not exist
import at.cyan.smartprotection.internet.UrlCheckResult;
                                       ^
../base/android/java/src/org/chromium/base/BaseChromiumApplication.java:24: error: package at.cyan.smartprotection.shared does not exist
import at.cyan.smartprotection.shared.ControlUnit;
                                     ^
../base/android/java/src/org/chromium/base/BaseChromiumApplication.java:42: error: cannot find symbol

    private final ControlUnit controlUnit;
                  ^
  symbol:   class ControlUnit
  location: class BaseChromiumApplication
3 errors

ninja: build stopped: subcommand failed.

Dne čtvrtek 17. září 2015 14:38:20 UTC+2 d...@chromium.org napsal(a):

[Anton Vayvod]

Could you check if proguard is run on the JAR files you added? The processed JAR files should be in out/{Debug|Release}/lib.java/

third_party/android_tools/proguard.flags basically contains a whitelist of what Chrome is using from the android_tools repository. You may need to add the packages of your libraries there.

To unsubscribe from this group and stop receiving emails from it, send an email to chromium-dev...@chromium.org.

[Vladimir Svoboda]

Yes, libraries are at out/Release/lib.java/ here's picture:https://drive.google.com/file/d/0Bxloda0H7AL2UFg4ZGtHSW5fYkE/view?usp=sharing

I've tryed to add this to progruard.flags: -keep class at.cyan.smartprotection.**,...;

But error is the same + some proguard output:

./base/android/java/src/org/chromium/base/BaseChromiumApplication.java:23: error: package at.cyan.smartprotection.internet does not exist
import at.cyan.smartprotection.internet.UrlCheckResult;
                                       ^
../base/android/java/src/org/chromium/base/BaseChromiumApplication.java:24: error: package at.cyan.smartprotection.shared does not exist
import at.cyan.smartprotection.shared.ControlUnit;
                                     ^
../base/android/java/src/org/chromium/base/BaseChromiumApplication.java:42: error: cannot find symbol
    private final ControlUnit controlUnit;
                  ^
  symbol:   class ControlUnit
  location: class BaseChromiumApplication
3 errors

ProGuard, version 4.7
Reading program jar [/home/lada/diplomka/chromium/src/third_party/android_tools/sdk/extras/google/google_play_services/libproject/google-play-services_lib/libs/google-play-services.jar]
Reading library jar [/home/lada/diplomka/chromium/src/out/Release/lib.java/android_support_v7_appcompat_javalib.jar]
Reading library jar [/home/lada/diplomka/chromium/src/third_party/android_tools/sdk/extras/android/support/v13/android-support-v13.jar]
Reading library jar [/home/lada/diplomka/chromium/src/out/Release/lib.java/android_support_v7_mediarouter_javalib.jar]
Reading library jar [/home/lada/diplomka/chromium/src/third_party/android_tools/sdk/platforms/android-22/android.jar]
Reading library jar [/home/lada/diplomka/chromium/src/third_party/android_tools/sdk/extras/android/support/v7/mediarouter/libs/android-support-v7-mediarouter.jar]
Reading library jar [/home/lada/diplomka/chromium/src/third_party/android_tools/sdk/extras/android/support/v7/appcompat/libs/android-support-v7-appcompat.jar]
Note: com.google.android.gms.ads.internal.purchase.zzb accesses a declared method 'asInterface(android.os.IBinder)' dynamically
      Maybe this is library method 'android.support.v4.app.INotificationSideChannel$Stub { android.support.v4.app.INotificationSideChannel asInterface(android.os.IBinder); }'
      Maybe this is library method 'android.support.v4.media.session.IMediaControllerCallback$Stub { android.support.v4.media.session.IMediaControllerCallback asInterface(android.os.IBinder); }'
      Maybe this is library method 'android.support.v4.media.session.IMediaSession$Stub { android.support.v4.media.session.IMediaSession asInterface(android.os.IBinder); }'
Note: the configuration keeps the entry point 'com.google.android.gms.common.api.zze$zza { void zza(com.google.android.gms.common.ConnectionResult,com.google.android.gms.signin.internal.AuthAccountResult); }', but not the descriptor class 'com.google.android.gms.signin.internal.AuthAccountResult'
Note: the configuration keeps the entry point 'com.google.android.gms.signin.internal.zzb { void zza(com.google.android.gms.common.ConnectionResult,com.google.android.gms.signin.internal.AuthAccountResult); }', but not the descriptor class 'com.google.android.gms.signin.internal.AuthAccountResult'
Note: there were 2 unkept descriptor classes in kept class members.
      You should consider explicitly keeping the mentioned classes
      (using '-keep').
Note: there were 1 accesses to class members by means of introspection.
      You should consider explicitly keeping the mentioned class members
      (using '-keep' or '-keepclassmembers').
Preparing output jar [/home/lada/diplomka/chromium/src/out/Release/gen/google_play_services_default_javalib_no_res/google_play_services_default_javalib_no_res.pre.jar]
  Copying resources from program jar [/home/lada/diplomka/chromium/src/third_party/android_tools/sdk/extras/google/google_play_services/libproject/google-play-services_lib/libs/google-play-services.jar]

ninja: build stopped: subcommand failed.

Dne čtvrtek 17. září 2015 16:24:37 UTC+2 Anton Vayvod napsal(a):

...

[Torne (Richard Coles)]

On Thu, 17 Sep 2015 at 13:43 d...@chromium.org <d...@chromium.org> wrote:

My understanding of GYP is that targets do nothing by themselves. the includes parameter describes how your arguments should be used to build the various target you are looking for, So includes declares what type your target is. Right now, you just set variables but didn't do anything with them. So that's most likely one of the reasons why the jars were not picked up. Hopefully that's also the only reason.

More specifically, targets of type "none" do nothing by themselves, and simply rely on definitions of "actions" "rules" and "copies" inside the target. These are often provided by including a gypi file as they are often redundant, but they can also be specified in the target itself. Of course, here, it wasn't doing anything (and for various reasons gyp doesn't consider it an error for a target to have no actions).

Targets of other types *do* have default behaviour defined by gyp itself (compiling and/or linking, depending on type).

[Vladimir Svoboda]

It's working now, that JARs where created from AAR (only renaming the extension which someone suggested but it was not working, so I created JAR from AAR unzipping of that AAR and rename classes.jar to mylib.jar).

Once more that you guys.

Dne čtvrtek 17. září 2015 16:43:56 UTC+2 Vladimir Svoboda napsal(a):

...

[Tuấn Bùi Sĩ]

Vào 16:13:05 UTC+7 Thứ Sáu, ngày 11 tháng 9 năm 2015, Vladimir Svoboda đã viết: