You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by "kevinjqliu (via GitHub)" <gi...@apache.org> on 2025/10/07 02:34:52 UTC

[I] update rc.yml script [iceberg-cpp]

kevinjqliu opened a new issue, #254:
URL: https://github.com/apache/iceberg-cpp/issues/254

   I noticed that iceberg-cpp copied over the rc.yml script from iceberg-go. 
   https://github.com/apache/iceberg-cpp/blob/main/.github/workflows/rc.yml
   
   We just merged a fix in iceberg-go for rc.yml, https://github.com/apache/iceberg-go/pull/588 & https://github.com/apache/iceberg-go/pull/590
   
   Mainly, it address the issue where the folder in the tar.gz file contains the name with the -rc suffix.
   
   Currently:
   ```
   https://dist.apache.org/repos/dist/dev/iceberg/apache-iceberg-cpp-0.1.0-rc2/
     apache-iceberg-cpp-0.1.0.tar.gz
     apache-iceberg-cpp-0.1.0-rc2/ (untar)
   ```
   
   After fix:
   ```
     apache-iceberg-cpp-0.1.0.tar.gz
     apache-iceberg-cpp-0.1.0/ (untar)
   ```
   
   the fix _should_ transfer over directly, but needs to be verified :) 
   


-- 
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: issues-unsubscribe@iceberg.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


Re: [I] update rc.yml script [iceberg-cpp]

Posted by "kevinjqliu (via GitHub)" <gi...@apache.org>.
kevinjqliu commented on issue #254:
URL: https://github.com/apache/iceberg-cpp/issues/254#issuecomment-3377144975

   I checked against the recent release on apache svn, and still see the issue
   ```
   https://dist.apache.org/repos/dist/dev/iceberg/apache-iceberg-cpp-0.1.0-rc2/
     apache-iceberg-cpp-0.1.0.tar.gz
     apache-iceberg-cpp-0.1.0-rc2 (untar)
   ```
   
   something to be aware of for the next release. 
   
   merge https://github.com/apache/iceberg-go/pull/590 merges, i can backport the prs to this repo 


-- 
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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


Re: [I] update rc.yml script [iceberg-cpp]

Posted by "wgtmac (via GitHub)" <gi...@apache.org>.
wgtmac commented on issue #254:
URL: https://github.com/apache/iceberg-cpp/issues/254#issuecomment-3377158243

   Yes, we found the issue in the rc2 and then fixed it in the rc4 which became the final release.


-- 
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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


Re: [I] update rc.yml script [iceberg-cpp]

Posted by "kevinjqliu (via GitHub)" <gi...@apache.org>.
kevinjqliu closed issue #254: update rc.yml script
URL: https://github.com/apache/iceberg-cpp/issues/254


-- 
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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


Re: [I] update rc.yml script [iceberg-cpp]

Posted by "wgtmac (via GitHub)" <gi...@apache.org>.
wgtmac commented on issue #254:
URL: https://github.com/apache/iceberg-cpp/issues/254#issuecomment-3376085005

   I think we have addressed these issues during the 1st release of iceberg-cpp but it would be good if we double check that it works as expected.


-- 
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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


Re: [I] update rc.yml script [iceberg-cpp]

Posted by "kevinjqliu (via GitHub)" <gi...@apache.org>.
kevinjqliu commented on issue #254:
URL: https://github.com/apache/iceberg-cpp/issues/254#issuecomment-3377550579

   nice! i see its fixed in https://dist.apache.org/repos/dist/release/iceberg/apache-iceberg-cpp-0.1.0/


-- 
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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org