Skip to content

Conversation

@N6REJ
Copy link
Collaborator

@N6REJ N6REJ commented Aug 21, 2025

PR Type

Other


Description

  • Archive old MySQL versions to archived folder

  • Remove deprecated default_authentication_plugin from configurations

  • Add innodb_redo_log_capacity for MySQL 8.4+ versions

  • Update release bundle to August 2025


Diagram Walkthrough

flowchart LR
  A["Old MySQL Versions"] --> B["Archived Folder"]
  C["MySQL Configs"] --> D["Remove deprecated_authentication_plugin"]
  C --> E["Add innodb_redo_log_capacity"]
  F["Release Bundle"] --> G["Update to 2025.8.21"]
Loading

File Walkthrough

Relevant files
Miscellaneous
16 files
init.bat
Remove MySQL 5.7.39 init script                                                   
+0/-3     
init.bat
Remove MySQL 8.0.27 init script                                                   
+0/-3     
init.bat
Remove MySQL 8.0.29 init script                                                   
+0/-3     
init.bat
Remove MySQL 8.0.36 init script                                                   
+0/-3     
init.bat
Remove MySQL 8.0.37 init script                                                   
+0/-3     
init.bat
Remove MySQL 8.0.41 init script                                                   
+0/-3     
init.bat
Remove MySQL 8.0.42 init script                                                   
+0/-3     
init.bat
Remove MySQL 8.3.0 init script                                                     
+0/-3     
my.ini
Remove entire MySQL 5.7.39 configuration                                 
+0/-53   
my.ini.ber
Remove entire MySQL 5.7.39 configuration backup                   
+0/-53   
bearsampp.conf
Remove MySQL 8.0.29 bearsampp configuration                           
+0/-10   
my.ini
Remove entire MySQL 8.0.29 configuration                                 
+0/-55   
my.ini.ber
Remove entire MySQL 8.0.29 configuration backup                   
+0/-55   
bearsampp.conf
Remove MySQL 8.0.30 bearsampp configuration                           
+0/-10   
bearsampp.conf
Remove MySQL 8.0.31 bearsampp configuration                           
+0/-10   
my.ini
Remove entire MySQL 8.0.31 configuration                                 
+0/-55   
Configuration changes
4 files
my.ini
Remove deprecated authentication plugin setting                   
+1/-1     
my.ini.ber
Remove deprecated authentication plugin setting                   
+1/-1     
my.ini
Remove deprecated authentication plugin setting                   
+1/-1     
my.ini.ber
Remove deprecated authentication plugin setting                   
+1/-1     
Additional files
38 files
bearsampp.conf [link]   
init.bat [link]   
bearsampp.conf [link]   
init.bat [link]   
my.ini.ber +0/-55   
bearsampp.conf +0/-10   
my.ini +0/-55   
my.ini.ber +0/-55   
bearsampp.conf +0/-10   
my.ini +0/-55   
my.ini.ber +0/-55   
bearsampp.conf +0/-10   
my.ini +0/-55   
my.ini.ber +0/-55   
bearsampp.conf +0/-10   
my.ini +0/-55   
my.ini.ber +0/-55   
bearsampp.conf +0/-10   
my.ini +0/-55   
my.ini.ber +0/-55   
bearsampp.conf +1/-1     
init.bat [link]   
my.ini +8/-6     
my.ini.ber +9/-6     
my.ini +1/-1     
my.ini.ber +1/-1     
my.ini +1/-1     
my.ini.ber +1/-1     
my.ini +1/-1     
my.ini.ber +1/-1     
my.ini +1/-1     
my.ini.ber +1/-1     
bearsampp.conf +1/-1     
init.bat [link]   
my.ini +11/-6   
my.ini.ber +11/-6   
build.properties +1/-1     
releases.properties +2/-0     

@N6REJ N6REJ added the enhancement ✨ Improve program label Aug 21, 2025
@qodo-merge-pro
Copy link

qodo-merge-pro bot commented Aug 21, 2025

PR Reviewer Guide 🔍

(Review updated until commit c3a422a)

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Recommended focus areas for review

Config Comment Clarity

The inline comments reference 'In bin/mysql9.4.0/my.ini' within the 8.4.6 config, which could confuse maintainers. Ensure comments accurately reflect the current file/version context.

# In bin/mysql9.4.0/my.ini
# (skip-federated and innodb_log_file_size are removed in MySQL 8.4+,
# so they should be deleted, not commented out.
# innodb_redo_log_capacity should be used instead of innodb_log_file_size if needed)
innodb_redo_log_capacity = 256M # Example value
Default Auth Note Wording

The added explanatory comments about authentication plugins are verbose and include guidance text. Consider tightening or moving this to documentation to keep config minimal while still noting that mysql_native_password is unsupported in 9.x.

sql_mode=NO_ENGINE_SUBSTITUTION,STRICT_TRANS_TABLES,NO_ZERO_IN_DATE,NO_ZERO_DATE,ERROR_FOR_DIVISION_BY_ZERO

# mysql-native-password setting removed for MySQL 9.4.0 (unknown variable in 9.x)
# Note: default_authentication_plugin=mysql_native_password is NOT valid in MySQL 9.x (plugin removed). Leave it unset to use the default caching_sha2_password.
character-set-server=UTF8MB4
Example Value Hardcoded

'innodb_redo_log_capacity = 256M # Example value' sets a potentially suboptimal default. Consider making it commented with guidance, or choosing a value aligned with expected data volume.

innodb_redo_log_capacity = 256M # Example value

@qodo-merge-pro
Copy link

qodo-merge-pro bot commented Aug 21, 2025

PR Code Suggestions ✨

Latest suggestions up to c3a422a

CategorySuggestion                                                                                                                                    Impact
Possible issue
Remove deprecated SQL modes

Remove the deprecated NO_ZERO_DATE and NO_ZERO_IN_DATE flags from the sql_mode
setting in the MySQL 9.4.0 configuration to prevent server startup failures.

bin/mysql9.4.0/my.ini [33-37]

-sql_mode=NO_ENGINE_SUBSTITUTION,STRICT_TRANS_TABLES,NO_ZERO_IN_DATE,NO_ZERO_DATE,ERROR_FOR_DIVISION_BY_ZERO
+sql_mode=NO_ENGINE_SUBSTITUTION,STRICT_TRANS_TABLES,ERROR_FOR_DIVISION_BY_ZERO
 ...
 character-set-server=UTF8MB4

[To ensure code accuracy, apply this suggestion manually]

Suggestion importance[1-10]: 9

__

Why: This suggestion correctly identifies that NO_ZERO_DATE and NO_ZERO_IN_DATE are removed in MySQL 9.0, and leaving them would cause a server startup failure, which is a critical bug.

High
Prevent uninitialized data directory failures

Ensure the MySQL data directory is initialized for new installations, as the
init.bat file for version 8.4.6 is empty and will not perform this step
automatically.

bin/mysql8.4.6/my.ini [18]

 datadir=~BEARSAMPP_LIN_PATH~/bin/mysql/mysql8.4.6/data
+# Note: Data directory must be initialized prior to first start (mysqld --initialize-insecure).
  • Apply / Chat
Suggestion importance[1-10]: 7

__

Why: The suggestion correctly points out that the init.bat file for MySQL 8.4.6 is empty, which could lead to startup failures if the data directory is not initialized, and asks the user to verify this.

Medium
General
Correct incorrect version reference

Correct the file path in a comment within bin/mysql8.4.6/my.ini from mysql9.4.0
to mysql8.4.6 to avoid confusion.

bin/mysql8.4.6/my.ini [26-30]

-# In bin/mysql9.4.0/my.ini
+# In bin/mysql8.4.6/my.ini
 # (skip-federated and innodb_log_file_size are removed in MySQL 8.4+,
 # so they should be deleted, not commented out.
 # innodb_redo_log_capacity should be used instead of innodb_log_file_size if needed)
 innodb_redo_log_capacity = 256M # Example value
  • Apply / Chat
Suggestion importance[1-10]: 4

__

Why: The suggestion correctly identifies a copy-paste error in a comment, which improves code clarity and maintainability.

Low
  • More

Previous suggestions

✅ Suggestions up to commit 6fd3aa7
CategorySuggestion                                                                                                                                    Impact
Possible issue
Remove invalid auth option
Suggestion Impact:The commit removed the mysql-native-password=ON line from the configuration.

code diff:

-mysql-native-password=ON
 character-set-server=UTF8MB4

Remove the invalid mysql-native-password=ON option from the my.ini file for
MySQL 8.4.6. This option is not a valid server variable and will cause a startup
failure.

bin/mysql8.4.6/my.ini [23-35]

 innodb_log_buffer_size = 8M
 #innodb_flush_log_at_trx_commit = 1
 #innodb_lock_wait_timeout = 50
-# In bin/mysql9.4.0/my.ini
-# (skip-federated and innodb_log_file_size are removed in MySQL 8.4+,
-# so they should be deleted, not commented out.
+# (skip-federated and innodb_log_file_size are removed in MySQL 8.4+;
 # innodb_redo_log_capacity should be used instead of innodb_log_file_size if needed)
-innodb_redo_log_capacity = 10M # Example value
+innodb_redo_log_capacity = 10M
 
 sql_mode=NO_ENGINE_SUBSTITUTION,STRICT_TRANS_TABLES,NO_ZERO_IN_DATE,NO_ZERO_DATE,ERROR_FOR_DIVISION_BY_ZERO
 
-mysql-native-password=ON
 character-set-server=UTF8MB4

[Suggestion processed]

Suggestion importance[1-10]: 9

__

Why: The suggestion correctly identifies that mysql-native-password=ON is an invalid configuration option that would cause the MySQL server to fail on startup, which is a critical bug.

High
Remove unsupported auth option
Suggestion Impact:The commit removed the mysql-native-password=ON line as suggested, though it also changed innodb_redo_log_capacity to 256M rather than just removing the comment.

code diff:

-innodb_redo_log_capacity = 10M # Example value
+innodb_redo_log_capacity = 256M # Example value
 
 sql_mode=NO_ENGINE_SUBSTITUTION,STRICT_TRANS_TABLES,NO_ZERO_IN_DATE,NO_ZERO_DATE,ERROR_FOR_DIVISION_BY_ZERO
 
-mysql-native-password=ON
 character-set-server=UTF8MB4

Remove the invalid mysql-native-password=ON option from the my.ini.ber file for
MySQL 8.4.6. This option is not a valid server variable and will cause a startup
failure.

bin/mysql8.4.6/my.ini.ber [27-36]

-# In bin/mysql9.4.0/my.ini
-# (skip-federated and innodb_log_file_size are removed in MySQL 8.4+,
-# so they should be deleted, not commented out.
+# (skip-federated and innodb_log_file_size are removed in MySQL 8.4+;
 # innodb_redo_log_capacity should be used instead of innodb_log_file_size if needed)
-innodb_redo_log_capacity = 10M # Example value
+innodb_redo_log_capacity = 10M
 
 sql_mode=NO_ENGINE_SUBSTITUTION,STRICT_TRANS_TABLES,NO_ZERO_IN_DATE,NO_ZERO_DATE,ERROR_FOR_DIVISION_BY_ZERO
 
-mysql-native-password=ON
 character-set-server=UTF8MB4

[Suggestion processed]

Suggestion importance[1-10]: 9

__

Why: The suggestion correctly identifies that mysql-native-password=ON is an invalid configuration option that would cause the MySQL server to fail on startup, which is a critical bug in the .ber config file.

High
General
Avoid unsafe tiny redo log
Suggestion Impact:The commit changed innodb_redo_log_capacity from 10M to 256M, aligning with the suggestion to avoid a tiny redo log.

code diff:

-innodb_redo_log_capacity = 10M # Example value
+innodb_redo_log_capacity = 256M # Example value

Increase the value for innodb_redo_log_capacity from 10M to a safer default,
such as 100M or higher, to avoid potential performance or crash-recovery issues.

bin/mysql9.4.0/my.ini [27-31]

-# In bin/mysql9.4.0/my.ini
-# (skip-federated and innodb_log_file_size are removed in MySQL 8.4+,
-# so they should be deleted, not commented out.
-# innodb_redo_log_capacity should be used instead of innodb_log_file_size if needed)
-innodb_redo_log_capacity = 10M # Example value
+# InnoDB redo log capacity (use bytes or suffixes like M/G). Commented to use MySQL default unless explicitly tuned.
+#innodb_redo_log_capacity = 256M
Suggestion importance[1-10]: 7

__

Why: The suggestion correctly points out that the 10M value for innodb_redo_log_capacity is very low and could lead to performance issues, recommending a more sensible default.

Medium
✅ Suggestions up to commit 6fd3aa7
CategorySuggestion                                                                                                                                    Impact
Possible issue
Remove invalid auth variable
Suggestion Impact:The commit removed the invalid 'mysql-native-password=ON' line from my.ini, aligning with the suggestion's intent, though it did not add the suggested comment lines.

code diff:

-mysql-native-password=ON
 character-set-server=UTF8MB4

Remove the invalid configuration mysql-native-password=ON from
bin/mysql8.4.6/my.ini, as it will cause the server to fail on startup.

bin/mysql8.4.6/my.ini [34]

-mysql-native-password=ON
+# Use default auth (caching_sha2_password) in MySQL 8.4+
+# To force native password (not recommended), uncomment the next line:
+# default_authentication_plugin = mysql_native_password
Suggestion importance[1-10]: 9

__

Why: The suggestion correctly identifies that mysql-native-password=ON is an invalid configuration variable that would prevent the MySQL server from starting, which is a critical bug.

High
Fix wrong comment and unsafe redo size
Suggestion Impact:The redo log capacity was increased from 10M to 256M (not 512M), addressing the unsafe low value. The comment line change is not shown in the diff, but the capacity increase aligns with the suggestion’s intent.

code diff:

-innodb_redo_log_capacity = 10M # Example value
+innodb_redo_log_capacity = 256M # Example value

In bin/mysql8.4.6/my.ini, correct the comment to reference the current file and
increase the innodb_redo_log_capacity from 10M to a safer value like 512M to
prevent performance degradation.

bin/mysql8.4.6/my.ini [26-30]

-# In bin/mysql9.4.0/my.ini
+# In bin/mysql8.4.6/my.ini
 # (skip-federated and innodb_log_file_size are removed in MySQL 8.4+,
 # so they should be deleted, not commented out.
 # innodb_redo_log_capacity should be used instead of innodb_log_file_size if needed)
-innodb_redo_log_capacity = 10M # Example value
+innodb_redo_log_capacity = 512M
Suggestion importance[1-10]: 7

__

Why: The suggestion correctly identifies a misleading comment and a very low innodb_redo_log_capacity value, which could negatively impact performance.

Medium
General
Raise unsafe redo capacity
Suggestion Impact:The commit increased innodb_redo_log_capacity from 10M to 256M, addressing the concern of the value being too low (though not the exact 512M suggested).

code diff:

-innodb_redo_log_capacity = 10M # Example value
+innodb_redo_log_capacity = 256M # Example value

In bin/mysql9.4.0/my.ini, increase the innodb_redo_log_capacity from 10M to a
more reasonable value, such as 512M, to prevent potential instability and
performance problems.

bin/mysql9.4.0/my.ini [27-31]

-# In bin/mysql9.4.0/my.ini
-# (skip-federated and innodb_log_file_size are removed in MySQL 8.4+,
-# so they should be deleted, not commented out.
-# innodb_redo_log_capacity should be used instead of innodb_log_file_size if needed)
-innodb_redo_log_capacity = 10M # Example value
+# innodb_redo_log_capacity controls total redo log size; adjust per workload.
+innodb_redo_log_capacity = 512M
Suggestion importance[1-10]: 7

__

Why: The suggestion correctly points out that the innodb_redo_log_capacity is set to a very low value of 10M, which could cause performance issues under load.

Medium
✅ Suggestions up to commit 56931c0
CategorySuggestion                                                                                                                                    Impact
Possible issue
Remove invalid option name
Suggestion Impact:The commit removed the invalid line "mysql-native-password=ON" from the configuration, addressing the suggestion.

code diff:

 sql_mode=NO_ENGINE_SUBSTITUTION,STRICT_TRANS_TABLES,NO_ZERO_IN_DATE,NO_ZERO_DATE,ERROR_FOR_DIVISION_BY_ZERO
-default_authentication_plugin=mysql_native_password
-mysql-native-password=ON
+
 character-set-server=UTF8MB4

Remove the invalid configuration option mysql-native-password=ON as it will
cause startup errors; rely on default_authentication_plugin instead.

bin/mysql8.4.6/my.ini [33-35]

 default_authentication_plugin=mysql_native_password
-mysql-native-password=ON
 character-set-server=UTF8MB4

[Suggestion processed]

Suggestion importance[1-10]: 9

__

Why: The suggestion correctly identifies that mysql-native-password=ON is an invalid configuration option that would cause the MySQL server to fail on startup, which is a critical bug.

High
Eliminate unsupported config option
Suggestion Impact:The commit removed the mysql-native-password=ON line and added a comment explaining its removal for MySQL 9.x.

code diff:

-default_authentication_plugin=mysql_native_password
-mysql-native-password=ON
+
+# mysql-native-password setting removed for MySQL 9.4.0 (unknown variable in 9.x)
 character-set-server=UTF8MB4

Remove the invalid configuration option mysql-native-password=ON as it will
cause startup errors; rely on default_authentication_plugin instead.

bin/mysql9.4.0/my.ini [34-36]

 default_authentication_plugin=mysql_native_password
-mysql-native-password=ON
 character-set-server=UTF8MB4

[Suggestion processed]

Suggestion importance[1-10]: 9

__

Why: The suggestion correctly identifies that mysql-native-password=ON is an invalid configuration option that would cause the MySQL server to fail on startup, which is a critical bug.

High
Suggestions up to commit acb31a5
CategorySuggestion                                                                                                                                    Impact
High-level
Incomplete/invalid MySQL configs

Several my.ini changes look inconsistent with upstream defaults and may break
startup: switching default_authentication_plugin to mysql-native-password=ON
(nonstandard directive) and commenting out critical options (skip-federated,
innodb_log_file_size) for 9.4.0 without justification. Also, new init.bat files
are empty, which removes database initialization behavior. Align directives with
the correct MySQL 8.4/9.4 syntax, restore required initialization scripts, and
verify version-specific options to ensure servers start reliably.

Examples:

bin/mysql8.4.6/my.ini [31]
mysql-native-password=ON
bin/mysql9.4.0/my.ini [20-25]
# skip-federated

server-id = 1

innodb_buffer_pool_size = 16M
# innodb_log_file_size = 5M

Solution Walkthrough:

Before:

# In bin/mysql8.4.6/my.ini
...
sql_mode=...
mysql-native-password=ON
character-set-server=UTF8MB4
...

# In bin/mysql9.4.0/my.ini
...
# skip-federated
...
# innodb_log_file_size = 5M
...

# init.bat for new versions is missing

After:

# In bin/mysql8.4.6/my.ini
...
sql_mode=...
default_authentication_plugin=mysql_native_password
character-set-server=UTF8MB4
...

# In bin/mysql9.4.0/my.ini
# (skip-federated and innodb_log_file_size are removed in MySQL 8.4+,
# so they should be deleted, not commented out.
# innodb_redo_log_capacity should be used instead of innodb_log_file_size if needed)
...
innodb_redo_log_capacity = 10M # Example value
...

# Add bin/mysql8.4.6/init.bat and bin/mysql9.4.0/init.bat
@ECHO OFF
%~dp0bin\mysqld.exe --initialize-insecure
Suggestion importance[1-10]: 9

__

Why: This suggestion correctly identifies multiple critical configuration errors and omissions, such as an invalid directive and missing initialization scripts, which would likely prevent the new MySQL versions from functioning correctly.

High

@jwaisner
Copy link
Contributor

@N6REJ , can you look at the recommendations provided by the AI. Looks like some config file changes are substantial and potentially breaking to how we have released in the past.

Comment on lines 34 to 36

# mysql-native-password setting removed for MySQL 9.4.0 (unknown variable in 9.x)
character-set-server=UTF8MB4

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggestion: Eliminate unsupported config option

Suggested change
# mysql-native-password setting removed for MySQL 9.4.0 (unknown variable in 9.x)
character-set-server=UTF8MB4
default_authentication_plugin=mysql_native_password
character-set-server=UTF8MB4

@qodo-merge-pro
Copy link

Persistent suggestions updated to latest commit 6fd3aa7

Comment on lines 25 to 35
#innodb_lock_wait_timeout = 50
# In bin/mysql9.4.0/my.ini
# (skip-federated and innodb_log_file_size are removed in MySQL 8.4+,
# so they should be deleted, not commented out.
# innodb_redo_log_capacity should be used instead of innodb_log_file_size if needed)
innodb_redo_log_capacity = 10M # Example value

sql_mode=NO_ENGINE_SUBSTITUTION,STRICT_TRANS_TABLES,NO_ZERO_IN_DATE,NO_ZERO_DATE,ERROR_FOR_DIVISION_BY_ZERO
default_authentication_plugin=mysql_native_password

mysql-native-password=ON
character-set-server=UTF8MB4

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggestion: Remove invalid auth option

Suggested change
#innodb_lock_wait_timeout = 50
# In bin/mysql9.4.0/my.ini
# (skip-federated and innodb_log_file_size are removed in MySQL 8.4+,
# so they should be deleted, not commented out.
# innodb_redo_log_capacity should be used instead of innodb_log_file_size if needed)
innodb_redo_log_capacity = 10M # Example value
sql_mode=NO_ENGINE_SUBSTITUTION,STRICT_TRANS_TABLES,NO_ZERO_IN_DATE,NO_ZERO_DATE,ERROR_FOR_DIVISION_BY_ZERO
default_authentication_plugin=mysql_native_password
mysql-native-password=ON
character-set-server=UTF8MB4
#innodb_lock_wait_timeout = 50
# (skip-federated and innodb_log_file_size are removed in MySQL 8.4+;
# innodb_redo_log_capacity should be used instead of innodb_log_file_size if needed)
innodb_redo_log_capacity = 10M
sql_mode=NO_ENGINE_SUBSTITUTION,STRICT_TRANS_TABLES,NO_ZERO_IN_DATE,NO_ZERO_DATE,ERROR_FOR_DIVISION_BY_ZERO
character-set-server=UTF8MB4

Comment on lines 27 to 36
# In bin/mysql9.4.0/my.ini
# (skip-federated and innodb_log_file_size are removed in MySQL 8.4+,
# so they should be deleted, not commented out.
# innodb_redo_log_capacity should be used instead of innodb_log_file_size if needed)
innodb_redo_log_capacity = 10M # Example value

sql_mode=NO_ENGINE_SUBSTITUTION,STRICT_TRANS_TABLES,NO_ZERO_IN_DATE,NO_ZERO_DATE,ERROR_FOR_DIVISION_BY_ZERO
default_authentication_plugin=mysql_native_password

mysql-native-password=ON
character-set-server=UTF8MB4

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggestion: Remove unsupported auth option

Suggested change
# In bin/mysql9.4.0/my.ini
# (skip-federated and innodb_log_file_size are removed in MySQL 8.4+,
# so they should be deleted, not commented out.
# innodb_redo_log_capacity should be used instead of innodb_log_file_size if needed)
innodb_redo_log_capacity = 10M # Example value
sql_mode=NO_ENGINE_SUBSTITUTION,STRICT_TRANS_TABLES,NO_ZERO_IN_DATE,NO_ZERO_DATE,ERROR_FOR_DIVISION_BY_ZERO
default_authentication_plugin=mysql_native_password
mysql-native-password=ON
character-set-server=UTF8MB4
# (skip-federated and innodb_log_file_size are removed in MySQL 8.4+;
# innodb_redo_log_capacity should be used instead of innodb_log_file_size if needed)
innodb_redo_log_capacity = 10M
sql_mode=NO_ENGINE_SUBSTITUTION,STRICT_TRANS_TABLES,NO_ZERO_IN_DATE,NO_ZERO_DATE,ERROR_FOR_DIVISION_BY_ZERO
character-set-server=UTF8MB4

…og_capacity to 256M and remove mysql-native-password option
@N6REJ
Copy link
Collaborator Author

N6REJ commented Sep 23, 2025

Ready to release

@N6REJ N6REJ merged commit 5d060c0 into main Sep 27, 2025
1 check passed
@N6REJ N6REJ deleted the august branch September 27, 2025 06:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants