Replaced all occurences of Charset.forName(UTF-8) with StringUtils.getPreferredCharset().#825
Conversation
|
Pending travis LGTM |
|
Travis failed due to a time-out. |
…tPreferredCharset().
|
LGTM, thanks @borisroman |
Replaced all occurences of Charset.forName(UTF-8) with StringUtils.getPreferredCharset(). * pr/825: Replaced all occurences of Charset.forName(UTF-8) with StringUtils.getPreferredCharset(). Signed-off-by: Daan Hoogland <daan@onecht.net>
|
Was this PR compiled locally? Master is now broken. When I try to build I get: BrocadeVcsApi.java:[378,26] error: cannot find symbol I checked the file and found out the following:
import com.cloud.network.schema.interfacevlan.InterfaceVlan;
import com.cloud.dc.Vlan; Which gives also this error: PortProfile() is not public in PortProfile; cannot be accessed from outside package. The imports are all wrong. Could you please fix it? ping @remibergsma @miguelaferreira @DaanHoogland @karuturi @bhaisaab Cheers, |
|
@wilderrodrigues If something breaks master, any committer can revert it in order to restore stability. I didn't verify yet, but feel free to revert and ping the dev-list about it. |
|
Reverted by commit 6e426fa |
|
Building is passing again: [INFO] ------------------------------------------------------------------------ |
This reverts commit a44e2bf, reversing changes made to 06cefaf. We are reverting the PR apache#825 because it breaks Master.
This reverts commit a44e2bf, reversing changes made to 06cefaf. We are reverting the PR apache#825 because it breaks Master.
This reverts commit a44e2bf, reversing changes made to 06cefaf. We are reverting the PR apache#825 because it breaks Master.
This reverts commit a44e2bf, reversing changes made to 06cefaf. We are reverting the PR apache#825 because it breaks Master.
No description provided.