profile
viewpoint
If you are wondering where the data of this site comes from, please visit https://api.github.com/users/fcamblor/events. GitMemory does not store any data, but only uses NGINX to cache data for a period of time. The idea behind GitMemory is simply to give users a better reading experience.
Frédéric Camblor fcamblor 4sh Villenave d'Ornon, France http://fcamblor.wordpress.com

fcamblor/crowd-store 2

App to manage a store among several users

fcamblor/backbone-fundamentals 1

A creative-commons book on Backbone.js for beginners and advanced users alike

fcamblor/AlertifyJS 0

A javascript framework for developing pretty browser dialogs and notifications.

fcamblor/angular-typescript-webpack 0

My angular typescript webpack architecture

fcamblor/angular2-codelab 0

Angular 2 Codelab: Let's develop an Angular 2 application...

fcamblor/awesome-java 0

A curated list of awesome Java frameworks, libraries and software. Inspired by awesome-python.

fcamblor/backbone-deep-model 0

Improved support for models with nested attributes.

create barnchrestx/restx

branch : dependabot/maven/jackson.version-2.12.1

created branch time in 14 days

PR opened restx/restx

Bump jackson.version from 2.10.1 to 2.12.1

Bumps jackson.version from 2.10.1 to 2.12.1. Updates jackson-module-jsonSchema from 2.10.1 to 2.12.1 <details> <summary>Commits</summary> <ul> <li><a href="https://github.com/FasterXML/jackson-module-jsonSchema/commit/73efe75e833d0cade6fa4d3e6af6f5d08dcee263"><code>73efe75</code></a> [maven-release-plugin] prepare release jackson-module-jsonSchema-2.12.1</li> <li><a href="https://github.com/FasterXML/jackson-module-jsonSchema/commit/a2447f1b481d46c05f58e949c6793230ece0b3ab"><code>a2447f1</code></a> prepare for 2.12.1 release</li> <li><a href="https://github.com/FasterXML/jackson-module-jsonSchema/commit/171dcccccf54091ee8b895a6c39f9313166089a5"><code>171dccc</code></a> ...</li> <li><a href="https://github.com/FasterXML/jackson-module-jsonSchema/commit/645e709c75d66353b02f6eedbe4379b16c4a1233"><code>645e709</code></a> [maven-release-plugin] prepare for next development iteration</li> <li><a href="https://github.com/FasterXML/jackson-module-jsonSchema/commit/ad97273ef77911a872149f1e9b775a016800c89e"><code>ad97273</code></a> [maven-release-plugin] prepare release jackson-module-jsonSchema-2.12.0</li> <li><a href="https://github.com/FasterXML/jackson-module-jsonSchema/commit/540842939e7631572b50ec92095e12921faefa11"><code>5408429</code></a> Prepare for 2.12.0 release</li> <li><a href="https://github.com/FasterXML/jackson-module-jsonSchema/commit/db8e1c8ab0ac82d24799737493d23b7f4bc19614"><code>db8e1c8</code></a> back to snapshots</li> <li><a href="https://github.com/FasterXML/jackson-module-jsonSchema/commit/9b14e547cfad385af3601c8e4120720c19ffc5e4"><code>9b14e54</code></a> [maven-release-plugin] prepare for next development iteration</li> <li><a href="https://github.com/FasterXML/jackson-module-jsonSchema/commit/72d7fa2a84c155acd3a81bbcca7e3ccbaef7a506"><code>72d7fa2</code></a> [maven-release-plugin] prepare release jackson-module-jsonSchema-2.12.0-rc2</li> <li><a href="https://github.com/FasterXML/jackson-module-jsonSchema/commit/a0c18bf7dbaa1226229e747c69537cc2066c27ac"><code>a0c18bf</code></a> prepare for 2.12.0-rc2</li> <li>Additional commits viewable in <a href="https://github.com/FasterXML/jackson-module-jsonSchema/compare/jackson-module-jsonSchema-2.10.1...jackson-module-jsonSchema-2.12.1">compare view</a></li> </ul> </details> <br />

Updates jackson-core from 2.10.1 to 2.12.1 <details> <summary>Commits</summary> <ul> <li><a href="https://github.com/FasterXML/jackson-core/commit/67a29707d4a41cbc52c25f1886eaf1da863f9c40"><code>67a2970</code></a> [maven-release-plugin] prepare release jackson-core-2.12.1</li> <li><a href="https://github.com/FasterXML/jackson-core/commit/2649dd44d64d39fb70bd45b1965c5832a3fc76bf"><code>2649dd4</code></a> Prepare for 2.12.1 release</li> <li><a href="https://github.com/FasterXML/jackson-core/commit/1e30c8c3122458660d7260099de0b04fbc88bc55"><code>1e30c8c</code></a> ...</li> <li><a href="https://github.com/FasterXML/jackson-core/commit/aa54f0a1b25b84fbdc50af69e71fdd40fcb65be1"><code>aa54f0a</code></a> javadocs, javadocs</li> <li><a href="https://github.com/FasterXML/jackson-core/commit/e5cbda207aa2b9e69da0f63bb79df5d9d072872b"><code>e5cbda2</code></a> More javadocing</li> <li><a href="https://github.com/FasterXML/jackson-core/commit/8dadf29d5b587a4f70eb0fa964a50a854a16327c"><code>8dadf29</code></a> Minor refactoring... and more javadoc stuff</li> <li><a href="https://github.com/FasterXML/jackson-core/commit/2493625b18d2e99f1747808c8e4d3d69e2629406"><code>2493625</code></a> Yet more javadoc'ing</li> <li><a href="https://github.com/FasterXML/jackson-core/commit/00a8c72eb4b56cfcfe1ea815cfcdcfbcc97b183e"><code>00a8c72</code></a> more javadoc stuff</li> <li><a href="https://github.com/FasterXML/jackson-core/commit/a40a7f7b80207f0c5c3c3802aca30d8ba66a4190"><code>a40a7f7</code></a> yet more javadoc fixes</li> <li><a href="https://github.com/FasterXML/jackson-core/commit/01d1e8877a2cb90f8b88ed952420bce808d0bd5a"><code>01d1e88</code></a> More javadoc work</li> <li>Additional commits viewable in <a href="https://github.com/FasterXML/jackson-core/compare/jackson-core-2.10.1...jackson-core-2.12.1">compare view</a></li> </ul> </details> <br />

Updates jackson-annotations from 2.10.1 to 2.12.1 <details> <summary>Commits</summary> <ul> <li>See full diff in <a href="https://github.com/FasterXML/jackson/commits">compare view</a></li> </ul> </details> <br />

Updates jackson-databind from 2.10.1 to 2.12.1 <details> <summary>Commits</summary> <ul> <li>See full diff in <a href="https://github.com/FasterXML/jackson/commits">compare view</a></li> </ul> </details> <br />

Updates jackson-datatype-guava from 2.10.1 to 2.12.1 <details> <summary>Commits</summary> <ul> <li><a href="https://github.com/FasterXML/jackson-datatypes-collections/commit/14cdc7ca32a0603b6ac7f138ae96801ca4168653"><code>14cdc7c</code></a> [maven-release-plugin] prepare release jackson-datatypes-collections-2.12.1</li> <li><a href="https://github.com/FasterXML/jackson-datatypes-collections/commit/fb630d8f96e4b7dbd797daa01ab44c8a0271e91e"><code>fb630d8</code></a> prepare for 2.12.1 release</li> <li><a href="https://github.com/FasterXML/jackson-datatypes-collections/commit/78d90063e8b64b59f1add9b4bbe639f200f16ef5"><code>78d9006</code></a> Merge branch '2.11' into 2.12</li> <li><a href="https://github.com/FasterXML/jackson-datatypes-collections/commit/6c7241ba9fb6439d5b1c8aa65df19a57c7ddec20"><code>6c7241b</code></a> ...</li> <li><a href="https://github.com/FasterXML/jackson-datatypes-collections/commit/f81efb50ff7f03b1e171c4850924af087b775718"><code>f81efb5</code></a> [maven-release-plugin] prepare for next development iteration</li> <li><a href="https://github.com/FasterXML/jackson-datatypes-collections/commit/f20d85bb69e6eb4fbbddcf5dda79ade2a5ae07b8"><code>f20d85b</code></a> [maven-release-plugin] prepare release jackson-datatypes-collections-2.11.4</li> <li><a href="https://github.com/FasterXML/jackson-datatypes-collections/commit/8dea997acba449d84d7776499254745dc34c5376"><code>8dea997</code></a> prepare for 2.11.4 release</li> <li><a href="https://github.com/FasterXML/jackson-datatypes-collections/commit/86684ee7c5cf778772efc10807e19cf699b7d419"><code>86684ee</code></a> Merge branch '2.11' into 2.12</li> <li><a href="https://github.com/FasterXML/jackson-datatypes-collections/commit/a3297ebbdbdbc3af2cc276b8245ada4fddb16e81"><code>a3297eb</code></a> Add missing SPI metadata for eclipse collections module</li> <li><a href="https://github.com/FasterXML/jackson-datatypes-collections/commit/ce9333128bd7c166a68a56f0c4ab03cfa8229c64"><code>ce93331</code></a> Add missing LICENSE files for guava, eclipse-collections; fix refs in other 2</li> <li>Additional commits viewable in <a href="https://github.com/FasterXML/jackson-datatypes-collections/compare/jackson-datatypes-collections-2.10.1...jackson-datatypes-collections-2.12.1">compare view</a></li> </ul> </details> <br />

Updates jackson-datatype-joda from 2.10.1 to 2.12.1 <details> <summary>Commits</summary> <ul> <li><a href="https://github.com/FasterXML/jackson-datatype-joda/commit/1d490319cd8982ad952dced25330cb274c2fe186"><code>1d49031</code></a> [maven-release-plugin] prepare release jackson-datatype-joda-2.12.1</li> <li><a href="https://github.com/FasterXML/jackson-datatype-joda/commit/cd124aeaade55853f7051a5d7132511f63de0dda"><code>cd124ae</code></a> prepare for 2.12.1 release</li> <li><a href="https://github.com/FasterXML/jackson-datatype-joda/commit/3e9580bcfea496161c9b35ad2bf8c9418894750f"><code>3e9580b</code></a> Fix the one lgtm.com warning</li> <li><a href="https://github.com/FasterXML/jackson-datatype-joda/commit/2144ef9493a704fb47a07df46f276c6b6f36a339"><code>2144ef9</code></a> Update release notes wrt <a href="https://github-redirect.dependabot.com/FasterXML/jackson-datatype-joda/issues/120">#120</a></li> <li><a href="https://github.com/FasterXML/jackson-datatype-joda/commit/6f8bfd5bbc5fb89000b35dcd893c5047aa9183a0"><code>6f8bfd5</code></a> Cache formatter with offset parsed.</li> <li><a href="https://github.com/FasterXML/jackson-datatype-joda/commit/bd09cb64da4ea60ec1005ced41ab28e4840172de"><code>bd09cb6</code></a> Merge branch '2.11' into 2.12</li> <li><a href="https://github.com/FasterXML/jackson-datatype-joda/commit/d58093b6b8cbaa371eeff36af29b471b16acdbbb"><code>d58093b</code></a> ...</li> <li><a href="https://github.com/FasterXML/jackson-datatype-joda/commit/164246c4cf5c7c36145a8dab25073275fa1d4e8e"><code>164246c</code></a> [maven-release-plugin] prepare for next development iteration</li> <li><a href="https://github.com/FasterXML/jackson-datatype-joda/commit/f3c1f43e42a51e9aaad6908a6370f6c9d67f0c96"><code>f3c1f43</code></a> [maven-release-plugin] prepare release jackson-datatype-joda-2.11.4</li> <li><a href="https://github.com/FasterXML/jackson-datatype-joda/commit/47ddd7d7b4cb7dc213ce382387e5c0a23e02f6df"><code>47ddd7d</code></a> prepare for 2.11.4 release</li> <li>Additional commits viewable in <a href="https://github.com/FasterXML/jackson-datatype-joda/compare/jackson-datatype-joda-2.10.1...jackson-datatype-joda-2.12.1">compare view</a></li> </ul> </details> <br />

Updates jackson-datatype-jsr310 from 2.10.1 to 2.12.1

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


<details> <summary>Dependabot commands and options</summary> <br />

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
  • @dependabot use these labels will set the current labels as the default for future PRs for this repo and language
  • @dependabot use these reviewers will set the current reviewers as the default for future PRs for this repo and language
  • @dependabot use these assignees will set the current assignees as the default for future PRs for this repo and language
  • @dependabot use this milestone will set the current milestone as the default for future PRs for this repo and language

You can disable automated security fix PRs for this repo from the Security Alerts page.

</details>

+1 -1

0 comment

1 changed file

pr created time in 14 days

push eventjenkinsci/nodejs-plugin

Bjorn Monnens

commit sha f9766ce75d3aa5ccdb6cde7029a41ca127c74a72

Fix arm64 path resolver to support more recent version of NodeJS

view details

push time in 23 days

push eventjenkinsci/nodejs-plugin

Bjorn Monnens

commit sha 85b76d563193ab80bb52ca63d9d27f35a2018615

Fix arm64 path resolver to support more recent version of NodeJS

view details

push time in 23 days

push eventjenkinsci/nodejs-plugin

Bjorn Monnens

commit sha 73249e046f176099fce90e323841b21ee5aa2df3

adding arm64 more recent version of node (#41) Fix arm64 path resolver to support more recent version of NodeJS

view details

push time in 23 days

PR merged jenkinsci/nodejs-plugin

adding arm64 more recent version of node

For my local RPI4 ubuntu based docker I needed a more recent version of node. Installed the the plugin build with this branch and now I am able to use the Node 15.8.0. On the main plugin this failed as it only goes up to 12.

+1334 -2

2 comments

3 changed files

bjornmonnens

pr closed time in 23 days

Pull request review commentjenkinsci/nodejs-plugin

adding arm64 more recent version of node

-{"list": [

You need just to download the updated file from http://updates.jenkins-ci.org/updates/hudson.plugins.nodejs.tools.NodeJSInstaller.json and remove the non JSON part downloadService.post('hudson.plugins.nodejs.tools.NodeJSInstaller', from begin and ) from end without apply any formatting

bjornmonnens

comment created time in 23 days

Pull request review commentjenkinsci/nodejs-plugin

adding arm64 more recent version of node

-{"list": [

After couple of commits, the review should now be clear. Seems the idea does automatic indentation upon paste.

bjornmonnens

comment created time in 24 days

Pull request review commentjenkinsci/nodejs-plugin

adding arm64 more recent version of node

 public String resolvePathFor(String version, Platform platform, CPU cpu) {             arch = "x64";             break;         case arm64:+            if (nodeVersion.compareTo(new NodeJSVersion(8, 6, 0)) == 0 || nodeVersion.compareTo(new NodeJSVersion(4, 0, 0)) < 0) {

I removed the baseline. Kept the case arm64 and armv7l as seperate cases (although the body is the same)

bjornmonnens

comment created time in 24 days

Pull request review commentjenkinsci/nodejs-plugin

adding arm64 more recent version of node

 public String resolvePathFor(String version, Platform platform, CPU cpu) {             arch = "x64";             break;         case arm64:+            if (nodeVersion.compareTo(new NodeJSVersion(8, 6, 0)) == 0 || nodeVersion.compareTo(new NodeJSVersion(4, 0, 0)) < 0) {

Yes but the baseline was wrong. For arm64 https://nodejs.org/dist/v8.6.0/node-v8.6.0-linux-arm64.tar.gz

bjornmonnens

comment created time in 24 days

Pull request review commentjenkinsci/nodejs-plugin

adding arm64 more recent version of node

-{"list": [

reverted the json file to the previous version and added the entries. Might be good to reformat the file to correct the indentation

bjornmonnens

comment created time in 24 days

Pull request review commentjenkinsci/nodejs-plugin

adding arm64 more recent version of node

 public String resolvePathFor(String version, Platform platform, CPU cpu) {             arch = "x64";             break;         case arm64:+            if (nodeVersion.compareTo(new NodeJSVersion(8, 6, 0)) == 0 || nodeVersion.compareTo(new NodeJSVersion(4, 0, 0)) < 0) {

This was already part of the initial code base https://github.com/jenkinsci/nodejs-plugin/blob/master/src/main/java/jenkins/plugins/nodejs/tools/pathresolvers/LatestInstallerPathResolver.java#L118

bjornmonnens

comment created time in 24 days

Pull request review commentjenkinsci/nodejs-plugin

adding arm64 more recent version of node

 public String resolvePathFor(String version, Platform platform, CPU cpu) {             break;         case amd64:             if (platform == Platform.SUNOS && //-                    (new NodeJSVersionRange("[7, 7.5)").includes(nodeVersion) || nodeVersion.compareTo(new NodeJSVersion(0, 12, 18)) == 0)) {+                    (   new NodeJSVersionRange("[7, 7.5)").includes(nodeVersion) ||

Changes reverted (note that that version is not available as download)

bjornmonnens

comment created time in 24 days

Pull request review commentjenkinsci/nodejs-plugin

adding arm64 more recent version of node

-{"list": [

Too many changes due reformatting, i'm not able to review changes

bjornmonnens

comment created time in 24 days

Pull request review commentjenkinsci/nodejs-plugin

adding arm64 more recent version of node

 public String resolvePathFor(String version, Platform platform, CPU cpu) {             break;         case amd64:             if (platform == Platform.SUNOS && //-                    (new NodeJSVersionRange("[7, 7.5)").includes(nodeVersion) || nodeVersion.compareTo(new NodeJSVersion(0, 12, 18)) == 0)) {+                    (   new NodeJSVersionRange("[7, 7.5)").includes(nodeVersion) ||

Please revert changes not related to this PR

bjornmonnens

comment created time in 24 days

Pull request review commentjenkinsci/nodejs-plugin

adding arm64 more recent version of node

 public String resolvePathFor(String version, Platform platform, CPU cpu) {             arch = "x64";             break;         case arm64:+            if (nodeVersion.compareTo(new NodeJSVersion(8, 6, 0)) == 0 || nodeVersion.compareTo(new NodeJSVersion(4, 0, 0)) < 0) {

Why 8.6.0 is excluded from download?

bjornmonnens

comment created time in 24 days

pull request commentjenkinsci/nodejs-plugin

adding arm64 more recent version of node

I've updates the test suite to contain the 15.8.0 entry, test should be green

bjornmonnens

comment created time in 24 days

Pull request review commentjenkinsci/nodejs-plugin

adding arm64 more recent version of node

 public String resolvePathFor(String version, Platform platform, CPU cpu) {             arch = "x64";             break;         case arm64:+            if (nodeVersion.compareTo(new NodeJSVersion(15, 8, 1)) >= 0 || nodeVersion.compareTo(new NodeJSVersion(8, 6, 0)) == 0 || nodeVersion.compareTo(new NodeJSVersion(4, 0, 0)) < 0) {

I removed the upper bound of 15.8.1, so should automatically allow to download newer versions.

bjornmonnens

comment created time in 24 days

Pull request review commentjenkinsci/nodejs-plugin

adding arm64 more recent version of node

 public String resolvePathFor(String version, Platform platform, CPU cpu) {             arch = "x64";             break;         case arm64:+            if (nodeVersion.compareTo(new NodeJSVersion(15, 8, 1)) >= 0 || nodeVersion.compareTo(new NodeJSVersion(8, 6, 0)) == 0 || nodeVersion.compareTo(new NodeJSVersion(4, 0, 0)) < 0) {

No, but the existing code had an upper limit of 12, hence I assumed you were writing the code defensively

bjornmonnens

comment created time in 24 days

Pull request review commentjenkinsci/nodejs-plugin

adding arm64 more recent version of node

 public String resolvePathFor(String version, Platform platform, CPU cpu) {             arch = "x64";             break;         case arm64:+            if (nodeVersion.compareTo(new NodeJSVersion(15, 8, 1)) >= 0 || nodeVersion.compareTo(new NodeJSVersion(8, 6, 0)) == 0 || nodeVersion.compareTo(new NodeJSVersion(4, 0, 0)) < 0) {

It's not clear why you ensure the version is lower than 15.8.1 (actually is not released yet) ? Do you expect arm64 will no more supported by nodejs?

bjornmonnens

comment created time in 24 days

pull request commentjenkinsci/nodejs-plugin

adding arm64 more recent version of node

Could you provide additional test case in InstallerPathResolversTest ?

bjornmonnens

comment created time in 24 days

PR opened jenkinsci/nodejs-plugin

adding arm64 more recent version of node

For my local RPI4 ubuntu based docker I needed a more recent version of node. Installed the the plugin build with this branch and now I am able to use the Node 15.8.0. On the main plugin this failed as it only goes up to 12.

+8 -1

0 comment

1 changed file

pr created time in 24 days

PR opened fcamblor/ts-object-transformer

Propose to export used custom types

In some cases( my for example :> ) these types may be required to use, so just want to propose this changes

+4 -3

0 comment

1 changed file

pr created time in a month

fork F1NYA/ts-object-transformer

Simple transformObject() utility function allowing to transform an object to another in a type-safe way

fork in a month

startedfcamblor/oh-my-zsh-agnoster-fcamblor

started time in a month

pull request commentjenkinsci/nodejs-plugin

Modified the NodeJS Image URL evaluation logic.

Got it. Totally agree. BTW, would you mind deleting line 125 and 126 in that file for me if you make any change in the future? Forgive me for being such a neat freak.

shawnda761

comment created time in a month

pull request commentjenkinsci/nodejs-plugin

Modified the NodeJS Image URL evaluation logic.

just an example the Apple Silicon has been recogneze as aarch64 -> arm64 and platform MacOs -> darwin In this case you could (not official for what I know) use the linux arm64 binary. Maybe there will not be available a specific distribution node-v15.7.0-darwin-arm64.tar.gz available for a long time. The LatestInstallerPathResolver can map given as input arm64 darwin to the linux distribution node-v15.7.0-linux-arm64.tar.gz

I confirm that the switch in LatestInstallerPathResolver is wrong and handle arm64 like arm6l where the support has been removed from version >=12. The switch case for arm64 should handle if platform == linux and version >= 4 than it's ok

shawnda761

comment created time in a month

pull request commentjenkinsci/nodejs-plugin

Modified the NodeJS Image URL evaluation logic.

If there is error where some combo (CPU+platform) seems not be supported but should in LatestInstallerPathResolver than please fill a new defect (for this specific issue) and propose a patch for LatestInstallerPathResolver to respect the real supported distribution. Something that limitation could e also dictated that the distribution exists but does not work for sure. That behaviour has been inherited from previous mantainer (the creator of this plugin) and I would change it unless I see a good mantanaible alternative to support actuals scenarios

shawnda761

comment created time in a month

pull request commentjenkinsci/nodejs-plugin

Modified the NodeJS Image URL evaluation logic.

I agree that evaluate the connection is not a good idea. How about add some configuration items to deal with those version limitations to alter current hard coding. Actually, I did this change just because I always got some error when I tried to use several 12+ versions NodeJS in a Linux arm64 machine. So I read the code and noticed some limitations in it (LatestInstallerPathResolver.java around line 118).

shawnda761

comment created time in a month