Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Jan 12 2021 13:11
    FroMage closed #1897
  • Jan 12 2021 13:11
    FroMage commented #1897
  • Jan 12 2021 11:26
    baberrehman commented #1897
  • Jan 12 2021 10:43
    FroMage commented #1897
  • Jan 12 2021 07:24
    baberrehman edited #1897
  • Jan 12 2021 07:23
    baberrehman edited #1897
  • Jan 12 2021 07:22
    baberrehman opened #1897
  • Feb 08 2020 19:30

    jvasileff on release-1.3.4

    (compare)

  • Feb 08 2020 19:29

    jvasileff on release-1.3.4

    (compare)

  • Feb 08 2019 12:29

    davidfestal on master

    Add the ability to tune the mvn… (compare)

  • Feb 07 2019 22:05

    davidfestal on master

    Switch to the `Eclipse Ceylon` … (compare)

  • Dec 03 2018 20:24
    kingjon3377 edited #1896
  • Dec 03 2018 20:22
    kingjon3377 opened #1896
  • Sep 29 2018 11:55

    gavinking on master

    fix a cosmetic problem in Dark … (compare)

  • Sep 29 2018 10:58

    gavinking on master

    use isConstructor() method (compare)

  • Sep 28 2018 23:15

    gavinking on master

    update for latest mvn version (compare)

  • Sep 28 2018 23:14

    gavinking on master

    adapt to API changes in Ceylon … (compare)

  • Sep 28 2018 13:31

    davidfestal on master

    Add Eclipse Photon compatibilit… Add Java 9 compatibility (#1890… Fix various bugs to have the ID… (compare)

  • Sep 26 2018 06:56
    davidfestal assigned #1894
  • Sep 26 2018 06:55
    davidfestal labeled #1894
Stéphane Épardaud
@FroMage
well, then one of the IDEs did that
and the fact of the matter is that we can't recover from that
there's no "try again if there's a .missing" file option
there should be
Tako Schotanus
@quintesse
Well the idea was that those .missing files should only be created when the Herd explicitly tells us that the file doesn't exist. So the same way we don't have a "redownload" option (because artifacts never change) there should theoretically also not be a need for a "retry missing files" option. If they appeared without reason then that is a bug.
Tom Bentley
@tombentley
Well surely it's possible for someone to query for xyz, get Not Found as the result, and then for xyz to be published.
Tako Schotanus
@quintesse
@tombentley no, it doesn't do this if the module itself doesn't exist, only for individual artifacts within a module
(the cache isn't filled with mistyped module names for example)
Stéphane Épardaud
@FroMage
well, something did put those files, wrongly because herd always had them
so, until we make damn sure this can never happen, we'd better add an option to recheck them ;)
because I sure as hell did not manually put those files in
Tako Schotanus
@quintesse
Then we should just add an option to force a redownload of all files. Because we when files are incorrectly downloaded or are corrupted in some other way we also have a problem
Gavin King
@gavinking
@davidfestal the build of the Eclipse plugin seems to be broken
Bastien Jansen
@bjansen
err that stupid Jenkins hasn't built ceylon-ide-common for 28 days :(
Bastien Jansen
@bjansen
I'm working on the build problem, it looks like something changed in the dist that breaks how modules are queried when the module version is surrounded with quotes
Bastien Jansen
@bjansen
@FroMage the error was introduced in ceylon/ceylon@e1bcdf0
according to git bisect
Stéphane Épardaud
@FroMage
damnit
for which module?
do you know?
Bastien Jansen
@bjansen
wdym?
Stéphane Épardaud
@FroMage
well, which module does it cause issues for?
Bastien Jansen
@bjansen
it looks like versions were always quoted in the generated MANIFEST.MF
the Eclipse plugin
source/com/redhat/ceylon/eclipse/module.ceylon:20: error: cannot find module artifact 'ceylon.language-"1.3.2.osgi-4"(.car|.jar)'
    shared import com.redhat.ceylon.dist "1.3.2-SNAPSHOT";
    ^
      - dependency tree: 'com.redhat.ceylon.eclipse/1.3.2' -> 'com.redhat.ceylon.dist/1.3.2-SNAPSHOT' -> 'ceylon.language/"1.3.2.osgi-4"'
source/com/redhat/ceylon/eclipse/module.ceylon:20: error: cannot find module artifact 'com.redhat.ceylon.langtools.classfile-"1.3.2.osgi-4"(.car|.jar)'
    shared import com.redhat.ceylon.dist "1.3.2-SNAPSHOT";
    ^
      - dependency tree: 'com.redhat.ceylon.eclipse/1.3.2' -> 'com.redhat.ceylon.dist/1.3.2-SNAPSHOT' -> 'com.redhat.ceylon.langtools.classfile/"1.3.2.osgi-4"'
source/com/redhat/ceylon/eclipse/module.ceylon:20: error: cannot find module artifact 'ceylon.bootstrap-"1.3.2.osgi-4"(.car|.jar)'
    shared import com.redhat.ceylon.dist "1.3.2-SNAPSHOT";
    ^
      - dependency tree: 'com.redhat.ceylon.eclipse/1.3.2' -> 'com.redhat.ceylon.dist/1.3.2-SNAPSHOT' -> 'ceylon.bootstrap/"1.3.2.osgi-4"'
source/com/redhat/ceylon/eclipse/module.ceylon:20: error: cannot find module artifact 'com.redhat.ceylon.model-"1.3.2.osgi-4"(.car|.jar)'
    shared import com.redhat.ceylon.dist "1.3.2-SNAPSHOT";
    ^
      - dependency tree: 'com.redhat.ceylon.eclipse/1.3.2' -> 'com.redhat.ceylon.dist/1.3.2-SNAPSHOT' -> 'com.redhat.ceylon.model/"1.3.2.osgi-4"'
source/com/redhat/ceylon/eclipse/module.ceylon:20: error: cannot find module artifact 'com.redhat.ceylon.module-resolver-"1.3.2.osgi-4"(.car|.jar)'
    shared import com.redhat.ceylon.dist "1.3.2-SNAPSHOT";
    ^
      - dependency tree: 'com.redhat.ceylon.eclipse/1.3.2' -> 'com.redhat.ceylon.dist/1.3.2-SNAPSHOT' -> 'com.redhat.ceylon.module-resolver/"1.3.2.osgi-4"'
source/com/redhat/ceylon/eclipse/module.ceylon:20: error: cannot find module artifact 'com.redhat.ceylon.common-"1.3.2.osgi-4"(.car|.jar)'
    shared import com.redhat.ceylon.dist "1.3.2-SNAPSHOT";
    ^
      - dependency tree: 'com.redhat.ceylon.eclipse/1.3.2' -> 'com.redhat.ceylon.dist/1.3.2-SNAPSHOT' -> 'com.redhat.ceylon.common/"1.3.2.osgi-4"'
Note: Created module com.redhat.ceylon.eclipse/1.3.2
ceylon compile: There were 6 errors
Gavin King
@gavinking
why would one quote versions in a MANIFEST?
Stéphane Épardaud
@FroMage
this is something that kicks in for modules with a module.xml, and it adds their group/artifact id if there's a pom.xml in there
it doesn't do anything with quotes or version
Bastien Jansen
@bjansen
modules that import com.redhat.ceylon.dist "1.3.2-SNAPSHOT" fail
Stéphane Épardaud
@FroMage
oh, dist
that's one odd beast
Bastien Jansen
@bjansen
@gavinking it's allowed because sometimes it's a range like bundle-version="[1.0,1.1)"
well not in that particular case, but quotes are valid
Stéphane Épardaud
@FroMage
but still I don't know why that would change anything wrt versions
my code really only sets the artifact/group
Bastien Jansen
@bjansen
@FroMage I think I know a workaround, but that still doesn't explain why it's now failing
we have this code that replaces a range with a single version and removes quotes at the same time:
        requireBundle = requireBundle.replaceAll(";bundle-version=\"\\[([^,]+),[^,]+(\\]|\\))\"", ";bundle-version=$1");
we could do something similar right after to remove quotes that are still present
in com.redhat.ceylon.cmr.impl.OSGiDependencyResolver#parseRequireBundle
Stéphane Épardaud
@FroMage
so the one thing I wonder is why we're even hitting this resolver
oh ok it's a jar with Activator and that's it
and indeed an OSGi decl
I was thinking I could add the same augment to that resolver to set group/artifact but it doesn't have one so it wouldn't help
but yeah, parseAttribute looks bogus to me
it should deal with optional quotes
not that horrible regex
Bastien Jansen
@bjansen
maybe we're doing it wrong and we shouldn't put those quotes in the first place
Stéphane Épardaud
@FroMage
isn't this a brilliant website? https://www.osgi.org/bundle-headers-reference/
no links to the definitions