Skip to content

stored: use "archive_device_string" variable for archive device#744

Merged
pstorz merged 2 commits intomasterfrom
dev/pstorz/use-archive_device-variable
Mar 12, 2021
Merged

stored: use "archive_device_string" variable for archive device#744
pstorz merged 2 commits intomasterfrom
dev/pstorz/use-archive_device-variable

Conversation

@pstorz
Copy link
Member

@pstorz pstorz commented Feb 23, 2021

The "archive device" directive was stored in the variable "device_name"
in the "DeviceResource" Class and in "dev_name" in the "Device" Class.

This is misleading as the device name (i.e. the name of the device
resource) is stored in the "resource_name_" member of the device resource.

Please follow these few prerequisites before you hand in a PR

Consider the following chapters in the Bareos Developer Documentation

Add some information

  • Add a small description to the CHANGELOG.md file and refer to your PR using this syntax '[PR #xyz]'
  • Add your name to the AUTHORS file

Keep spirit!

  • Do not be afraid to hand in a PR!

@pstorz pstorz force-pushed the dev/pstorz/use-archive_device-variable branch from 07b84c9 to 9fa7144 Compare February 23, 2021 16:56
@pstorz pstorz self-assigned this Feb 25, 2021
@pstorz pstorz force-pushed the dev/pstorz/use-archive_device-variable branch 2 times, most recently from 5d0dbad to 20f390a Compare February 25, 2021 09:52
@pstorz pstorz requested a review from franku March 9, 2021 09:44
Copy link
Contributor

@franku franku left a comment

Choose a reason for hiding this comment

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

Good idea to give a meaningful name..

However, please update the changelog.

@pstorz pstorz force-pushed the dev/pstorz/use-archive_device-variable branch 2 times, most recently from a257a24 to 64f2b95 Compare March 12, 2021 11:08
@pstorz pstorz changed the title stored: use "archive_device" variable for archive device stored: use "archive_device_string" variable for archive device Mar 12, 2021
pstorz added 2 commits March 12, 2021 12:59
The "archive device" directive was stored in the variable "device_name"
in the "DeviceResource" Class and in "dev_name" in the "Device" Class.

This is misleading as the device name (i.e. the name of the device
resource) is stored in the "resource_name_" member of the device resource.
@pstorz pstorz force-pushed the dev/pstorz/use-archive_device-variable branch from 64f2b95 to e1bccb5 Compare March 12, 2021 11:59
@pstorz pstorz merged commit fb76608 into master Mar 12, 2021
@pstorz pstorz deleted the dev/pstorz/use-archive_device-variable branch March 12, 2021 12:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants