You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by "JiriOndrusek (via GitHub)" <gi...@apache.org> on 2025/10/09 09:38:12 UTC
[PR] Upgade Camel to 4.15.0 [camel-upgrade-recipes]
JiriOndrusek opened a new pull request, #66:
URL: https://github.com/apache/camel-upgrade-recipes/pull/66
Migration to Camel 4.15.0
@Croway WDYT? there is a migration https://camel.apache.org/manual/camel-4x-upgrade-guide-4_15.html#_camel_netty_camel_netty_http, which addds prefix `file:` to property of the component. I should be able to migrate it automatically, but there might be some rare examples of usa, which might be too complex. (like a user, which sets this property programaticaly in java). Should I cover the base scenarios (java + yaml + xml DSL) and mark it as partially covered by migration tool?
As for CSB I don't think there is anything to take careof. Probably only update of the `latest` recipe
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Upgade Camel to 4.15.0 [camel-upgrade-recipes]
Posted by "JiriOndrusek (via GitHub)" <gi...@apache.org>.
JiriOndrusek merged PR #66:
URL: https://github.com/apache/camel-upgrade-recipes/pull/66
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Upgade Camel to 4.15.0 [camel-upgrade-recipes]
Posted by "davsclaus (via GitHub)" <gi...@apache.org>.
davsclaus commented on code in PR #66:
URL: https://github.com/apache/camel-upgrade-recipes/pull/66#discussion_r2416410359
##########
camel-spring-boot-upgrade-recipes/src/main/resources/META-INF/rewrite/4.15.yaml:
##########
@@ -0,0 +1,24 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license agreements. See the NOTICE file distributed with
+# this work for additional information regarding copyright ownership.
+# The ASF licenses this file to You under the Apache License, Version 2.0
+# (the "License"); you may not use this file except in compliance with
+# the License. You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+---
Review Comment:
This file says 4.14, should this not be 4.15 ?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Upgade Camel to 4.15.0 [camel-upgrade-recipes]
Posted by "Croway (via GitHub)" <gi...@apache.org>.
Croway commented on PR #66:
URL: https://github.com/apache/camel-upgrade-recipes/pull/66#issuecomment-3385121477
hey @JiriOndrusek thanks for looking into it.
From CSB pov, the creation of the 4.15.yaml and the update of the latest.yaml recipe is enough.
And yes, I checked the component, and the update of the properties has to be done on java setter (on the component), yaml endpoint properties, and xml endpoint properties
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Upgade Camel to 4.15.0 [camel-upgrade-recipes]
Posted by "JiriOndrusek (via GitHub)" <gi...@apache.org>.
JiriOndrusek commented on code in PR #66:
URL: https://github.com/apache/camel-upgrade-recipes/pull/66#discussion_r2416787923
##########
camel-spring-boot-upgrade-recipes/src/main/resources/META-INF/rewrite/4.15.yaml:
##########
@@ -0,0 +1,24 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license agreements. See the NOTICE file distributed with
+# this work for additional information regarding copyright ownership.
+# The ASF licenses this file to You under the Apache License, Version 2.0
+# (the "License"); you may not use this file except in compliance with
+# the License. You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+---
Review Comment:
my bad, I'll fix that
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Upgade Camel to 4.15.0 [camel-upgrade-recipes]
Posted by "JiriOndrusek (via GitHub)" <gi...@apache.org>.
JiriOndrusek commented on PR #66:
URL: https://github.com/apache/camel-upgrade-recipes/pull/66#issuecomment-3385893716
@Croway I added migrations of came and camel-http for the easiest cases,
but I kept java dsl unsupported, because the new method changes also type of the parameter (From java.util.File to String). I think that such change should be done manually by users.
I think that we can merge this and if all looks ok, I'd start new release tomorrow.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org