opengrok icon indicating copy to clipboard operation
opengrok copied to clipboard

address isWorking() for other repository types

Open vladak opened this issue 4 years ago • 9 comments

The fix for #3307 was done for GitRepository only. This bug tracks the rest of the repositories.

vladak avatar Mar 20 '21 22:03 vladak

Hi @vladak Can you please explain me about this issue really interested to work on this issue

gaurangkudale avatar Jan 12 '22 14:01 gaurangkudale

Explore the related issues and history. Specifically, for Git this was fixed in c47b7e716a31dc48f006bc51fc330c0b946affe3 (only to be removed by b289da3c) and for Mercurial in de458d8d. There are other types of SCMs under opengrok-indexer/src/main/java/org/opengrok/indexer/history/ that should receive the same treatment.

vladak avatar Jan 13 '22 10:01 vladak

Hi @vladak, Thank you for your response Can you please tell me more about this issue I'm trying to understand

gaurangkudale avatar Jan 13 '22 14:01 gaurangkudale

Hi @vladak, I'm willing to work on this issue, https://github.com/oracle/opengrok/tree/master/opengrok-indexer/src/main/java/org/opengrok/indexer/history In this tab, for which file is the problem araising for isWorking() function? can you specify the file, please.

Akash3121 avatar Mar 02 '23 10:03 Akash3121

Hey @vladak, https://github.com/oracle/opengrok/blob/03246776315da611de8587e903c16d2d6854a89c/opengrok-indexer/src/main/java/org/opengrok/indexer/history/MercurialRepository.java#L21 In this line, I think the year needs to be updated to 2023. please confirm. Thank you!

Akash3121 avatar Mar 02 '23 10:03 Akash3121

MercurialRepository is already addressed, see above.

vladak avatar Mar 02 '23 10:03 vladak

In which file does this problem arises, could you please specify one? Thank you!

Hey @vladak,

https://github.com/oracle/opengrok/blob/03246776315da611de8587e903c16d2d6854a89c/opengrok-indexer/src/main/java/org/opengrok/indexer/history/MercurialRepository.java#L21

In this line, I think the year needs to be updated to 2023. please confirm. Thank you!

please confirm regarding this one too!

Akash3121 avatar Mar 02 '23 11:03 Akash3121

In which file does this problem arises, could you please specify one? Thank you!

Basically in all Repository descendants that do not employ lazy loading of the working attribute.

Hey @vladak, https://github.com/oracle/opengrok/blob/03246776315da611de8587e903c16d2d6854a89c/opengrok-indexer/src/main/java/org/opengrok/indexer/history/MercurialRepository.java#L21

In this line, I think the year needs to be updated to 2023. please confirm. Thank you!

please confirm regarding this one too!

Looks fine to me. The file was last updated by Oracle employee in 2022.

vladak avatar Mar 02 '23 15:03 vladak

i am beginer any one guide me

DeepakKumarSingh27 avatar Apr 02 '23 08:04 DeepakKumarSingh27