git migration: Remove support for file versions

Git does not support the ability to replace a token with a version
string during check-in. While it does have support for replacing a
token on clone, this is somewhat sub-optimal: the token is replaced
with the object hash, which is not particularly easy for human
consumption. What's more, in practice, the source file version was often
not terribly useful. Generally, when triaging bugs, the overall version
of Asterisk is far more useful than an individual SVN version of a file.
As a result, this patch removes Asterisk's support for showing source file
versions.

Specifically, it does the following:
* main/asterisk:
  - Refactor the file_version structure to reflect that it no longer
    tracks a version field.
  - Alter the "core show file version" CLI command such that it always
    reports the version of Asterisk. The file version is no longer
    available.

* main/manager: The Version key now always reports the Asterisk version.

* UPGRADE: Add notes for:
  - Modification to the ModuleCheck AMI Action.
  - Modification of the "core show file version" CLI command.

Change-Id: Ia932d3c64cd18a14a3c894109baa657ec0a85d28
This commit is contained in:
Matt Jordan
2015-04-12 12:59:22 -05:00
parent 91c1ed7ef6
commit d1a6f1a9f9
4 changed files with 79 additions and 56 deletions

View File

@@ -159,6 +159,8 @@ int ast_shutdown_final(void);
* \param version the version string (typically a SVN revision keyword string)
* \return nothing
*
* \note As of 13.4.0, the \c version parameter is ignored.
*
* This function should not be called directly, but instead the
* ASTERISK_FILE_VERSION macro should be used to register a file with the core.
*/
@@ -175,12 +177,29 @@ void ast_register_file_version(const char *file, const char *version);
*/
void ast_unregister_file_version(const char *file);
/*! \brief Find version for given module name
/*!
* \brief Find version for given module name
* \param file Module name (i.e. chan_sip.so)
* \return version string or NULL if the module is not found
*
* \note As of 13.4.0, the file version is no longer tracked. As such,
* if the file exists, the Asterisk version will be returned.
*
* \retval NULL if the file doesn't exist.
* \retval The Asterisk version if the file does exist.
*/
const char *ast_file_version_find(const char *file);
/*!
* \brief Complete a source file name
* \param partial The partial name of the file to look up.
* \param n The n-th match to return.
*
* \retval NULL if there is no match for partial at the n-th position
* \retval Matching source file name
*
* \note A matching source file is allocataed on the heap, and must be
* free'd by the caller.
*/
char *ast_complete_source_filename(const char *partial, int n);
/*!